PR #298 mixes engineering logic with test-performance changes. Split it before merge so each review has one risk profile.
Proposed sequence:
- Add
coil_clearance.py and analytic geometry tests.
- Wire the clearance constraint into the port optimizer.
- Add plot/artifact generation in a separate PR if needed.
- Parallelize the chartmap wall-loss test separately.
Acceptance:
- Each PR states its risk tier and correctness contract.
- The OpenMP/test-performance change has no engineering behavior change.
- The engineering PRs can be reviewed without reading unrelated test scheduling changes.
PR #298 mixes engineering logic with test-performance changes. Split it before merge so each review has one risk profile.
Proposed sequence:
coil_clearance.pyand analytic geometry tests.Acceptance: