Skip to content

Copilot/Additinal examples - #5

Merged
QL-UoHull merged 7 commits into
mainfrom
copilot/paper-style-examples
Jul 16, 2026
Merged

QL-UoHull merged 7 commits into
mainfrom
copilot/paper-style-examples

Conversation

@QL-UoHull

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 16, 2026 15:08
@QL-UoHull

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Expands the repository’s usage demonstrations for the Python implicit-spline implementation by improving the introductory notebook’s robustness and significantly extending the standalone demo script to cover additional modeling scenarios (holes, partitions, multi-panel figures).

Changes:

  • Updated notebooks/01_basic_polygon.ipynb to use more robust local path discovery and to add/standardize several visualization examples (including compare_n and explicit 3D axes).
  • Refactored and expanded examples/demo.py into three scenario-based sections (convex polygons, holes, partitions) with additional helper visualizations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
notebooks/01_basic_polygon.ipynb Improves local import setup and enhances/standardizes plotting examples (adds compare_n, explicit 3D axes).
examples/demo.py Adds many additional demo scenarios (holes, partitions, multi-panel figures) and reorganizes the script accordingly.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread examples/demo.py
Comment thread examples/demo.py
QL-UoHull and others added 4 commits July 16, 2026 16:14
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@QL-UoHull

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved the merge conflicts in this PR in commit 3214a48 (includes merging latest main and branch updates).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants