feat(cam-clear): blend-corridor engagement re-certification #82

Open
patman wants to merge 1 commit from patman/clear-corridor into patman/blend-symmetric AGit
Owner

Retires #79's honest boundary "engagement certified on nominal path only". ClearParams gains blend_corridor: the plan's certified/reported engagement now covers ANY execution whose per-pass realized tool-center path stays within the corridor of nominal -- exactly what blend_pass enforces when called with max_deviation = blend_corridor (the corridor width does double duty with the blend deviation certificate).

Mechanism: every per-feature penetration in the certified-engagement machinery is evaluated at pen + 2c (c for the blended tool straying outward -- 1-Lipschitz; c for the prior pass's inward undercut leaving stock proud -- disk-inclusion), vertex blend-zone reach widens to r + 2c, and the Case-A/B innermost "proven zero" becomes an honest closed-form two-circle bound at penetration 2c (the inclusion proof only holds on the nominal path). Validation caps 2c <= 0.05 * tool_radius. Corridor 0 reproduces the nominal plan bitwise (pinned on the three standard fixtures).

Tests: bitwise-zero identity, monotonicity in c, flagship adversarial brute-force (independent StockState ground truth vs eroded-envelope stock + outward-pushed tool samples -- a missing margin would exceed the slack by ~3 orders), Case-A bump, validation, and an e2e in cam-drive (plan -> blend within budget -> drive_plan_fourth).

Caller contract (disclosed in cam-drive docs): the certificate holds only when blending uses max_deviation <= the planning corridor; RingPlan does not retain the corridor, so this is not runtime-enforced.

Adversarially reviewed (independent re-derivation of the Lipschitz/undercut/2c arguments, all 8 margin call sites audited, reach widening, sect. 3(b) survival, flagship independence + non-vacuity by magnitude): PASS; one MEDIUM doc-wording finding (certified_error invariance justification) corrected per the reviewer's exact wording.

Retires #79's honest boundary "engagement certified on nominal path only". `ClearParams` gains `blend_corridor`: the plan's certified/reported engagement now covers ANY execution whose per-pass realized tool-center path stays within the corridor of nominal -- exactly what `blend_pass` enforces when called with `max_deviation = blend_corridor` (the corridor width does double duty with the blend deviation certificate). Mechanism: every per-feature penetration in the certified-engagement machinery is evaluated at `pen + 2c` (c for the blended tool straying outward -- 1-Lipschitz; c for the prior pass's inward undercut leaving stock proud -- disk-inclusion), vertex blend-zone reach widens to `r + 2c`, and the Case-A/B innermost "proven zero" becomes an honest closed-form two-circle bound at penetration 2c (the inclusion proof only holds on the nominal path). Validation caps `2c <= 0.05 * tool_radius`. Corridor 0 reproduces the nominal plan bitwise (pinned on the three standard fixtures). Tests: bitwise-zero identity, monotonicity in c, flagship adversarial brute-force (independent StockState ground truth vs eroded-envelope stock + outward-pushed tool samples -- a missing margin would exceed the slack by ~3 orders), Case-A bump, validation, and an e2e in cam-drive (plan -> blend within budget -> drive_plan_fourth). Caller contract (disclosed in cam-drive docs): the certificate holds only when blending uses `max_deviation <= ` the planning corridor; `RingPlan` does not retain the corridor, so this is not runtime-enforced. Adversarially reviewed (independent re-derivation of the Lipschitz/undercut/2c arguments, all 8 margin call sites audited, reach widening, sect. 3(b) survival, flagship independence + non-vacuity by magnitude): PASS; one MEDIUM doc-wording finding (certified_error invariance justification) corrected per the reviewer's exact wording.
feat(cam-clear): blend-corridor engagement re-certification
Some checks are pending
CI / build-test (pull_request) Waiting to run
9652e21d7d
Co-authored-by: patman-assist <patrick-ai@kgroo.co>
Some checks are pending
CI / build-test (pull_request) Waiting to run
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin +refs/pull/82/head:patman/clear-corridor
git switch patman/clear-corridor

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch patman/blend-symmetric
git merge --no-ff patman/clear-corridor
git switch patman/clear-corridor
git rebase patman/blend-symmetric
git switch patman/blend-symmetric
git merge --ff-only patman/clear-corridor
git switch patman/clear-corridor
git rebase patman/blend-symmetric
git switch patman/blend-symmetric
git merge --no-ff patman/clear-corridor
git switch patman/blend-symmetric
git merge --squash patman/clear-corridor
git switch patman/blend-symmetric
git merge --ff-only patman/clear-corridor
git switch patman/blend-symmetric
git merge patman/clear-corridor
git push origin patman/blend-symmetric
Sign in to join this conversation.
No description provided.