Skip to content

Fix StaticArrays-backed polygons in clipping passthroughs - #532

Open
asinghvi17 wants to merge 1 commit into
mainfrom
fix/static-clipping-passthrough
Open

asinghvi17 wants to merge 1 commit into
mainfrom
fix/static-clipping-passthrough

Conversation

@asinghvi17

@asinghvi17 asinghvi17 commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Clipping and containment on StaticArrays-backed polygons have thrown a MethodError since #486. _fh_as_ring and _fh_as_poly built their lists with comprehensions, which give a SizedVector for static input, while the tracer's polygon type expects Vector storage. This allocates Vectors explicitly instead, and adds tests for intersection, union, difference and cut on SVector polygons on both manifolds. Found while chasing the ConservativeRegridding downstream failure. The one red OverlayNG locator test on main is unrelated; #530 fixes it.

🤖 Generated with Claude Code

Harness: codex
Model: gpt-6.1-sol

This branch has not been deployed

No deployments
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.

2 participants