patman/KGROO
1
1
Fork 0
Code Issues Pull requests 9 Projects Releases Packages Wiki Activity Actions 5

feat(cam-geom): add Contour::reversed for ring-to-island conversion #60

Merged
patman merged 1 commit from patman/cam-reversed into main 2026-07-09 23:27:53 +00:00 AGit
Conversation 0 Commits 1 Files changed 1 +146
patman commented 2026-07-04 17:24:36 +00:00
Owner
Copy link

Small primitive for the clearing strategy: ContourSegment::reversed + Contour::reversed (reverse segment order; lines swap endpoints; arcs start+=sweep, sweep negates). reversed.point_at(u) == original.point_at(1-u) exactly — reviewer derived the correspondence and confirmed every Contour::new invariant survives by symmetry, so re-validation is safely skipped. Needed to turn a CCW cleared-envelope ring into a CW stock island. Stacked on patman/cam-seed (#57).

Small primitive for the clearing strategy: ContourSegment::reversed + Contour::reversed (reverse segment order; lines swap endpoints; arcs start+=sweep, sweep negates). reversed.point_at(u) == original.point_at(1-u) exactly — reviewer derived the correspondence and confirmed every Contour::new invariant survives by symmetry, so re-validation is safely skipped. Needed to turn a CCW cleared-envelope ring into a CW stock island. Stacked on patman/cam-seed (#57).
Adaptive clearing (cam-clear) needs to turn a CCW tool-center ring into
a CW island contour for StockState; reversing winding by construction
(no re-validation) is a general contour primitive, so it lives here
rather than being duplicated in cam-clear.

Co-authored-by: patman-assist <patrick-ai@kgroo.co>
patman changed target branch from patman/cam-seed to main 2026-07-09 23:27:52 +00:00
patman merged commit 4668187cae into main 2026-07-09 23:27:53 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
AI Assisted

Built with AI assistance; Patrick is the primary author

  
Bug fix

Use on bug fixes

  
CAM

Computer aided manufacturing edits

  
Dirac

common/dirac math library

  
Docs

docs

  
Feature

Use on entirely new features

  
Infra

Provisioning, CI/CD, and dev tooling

  
Interpreter

Work on the interpreter core

  
Webfront

Static webfront only, not CAM interfaces or metrics

No labels
AI Assisted
Bug fix
CAM
Dirac
Docs
Feature
Infra
Interpreter
Webfront
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
patman/KGROO!60
No description provided.