Skip to content

feat(rate-of-closure): add advanced wedge impact visualization - #4179

Closed
dieterolson wants to merge 6 commits into
feat/4166-wedge-turf-physicsfrom
feat/4162-wedge-impact-visualization
Closed

feat(rate-of-closure): add advanced wedge impact visualization#4179
dieterolson wants to merge 6 commits into
feat/4166-wedge-turf-physicsfrom
feat/4162-wedge-impact-visualization

Conversation

@dieterolson

Copy link
Copy Markdown
Collaborator

Summary

  • evaluate retained pose, twist, and articulated wrist geometry at the exact impact/closest-approach time
  • add the versioned rate-of-closure.impact-scene/v1 UI-independent scene and metric contract
  • add locked-scale orbitable wedge impact views to PyQt6 and React with named cameras, vector toggles, keyboard/pointer controls, and PNG/SVG/JSON exports
  • make every web metric visibly expandable with equation, frame, units, assumptions, and availability
  • preserve miss labeling and explicit rigid-shaft, no-turf-feedback, and no-optimal-bounce claim boundaries

Verification

  • python -m pytest tests/rate_of_closure -q — 576 passed, one existing polynomial-generator legend warning
  • npm test -- --run — 347 passed
  • npm run build, npm run type-check, npm run lint — passed
  • Ruff check/format — passed
  • strict changed-module mypy — passed
  • protected module-size budget — passed
  • headless Chrome QA at 1600x1400 — named view, vector toggle, keyboard orbit, expanded definition; zero console errors
  • current standalone PyQt branch window launched and responsive

Stack

Base: #4178 (feat/4166-wedge-turf-physics). This PR must not merge before its stacked base and remains draft until dependency checks/reviews are complete.

Closes #4162
Part of #4158

@d-sorgcodexagent
d-sorgcodexagent Bot marked this pull request as ready for review August 11, 2026 22:10
@dieterolson

Copy link
Copy Markdown
Collaborator Author

Consolidated into #4410. Closing to eliminate CI runner load.

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.

1 participant