Skip to content

Fix acceleration limits and link velocity guard - #1

Merged
KohmeiK merged 1 commit into
masterfrom
codex/analytic-acceleration-validation
Jul 21, 2026
Merged

Fix acceleration limits and link velocity guard#1
KohmeiK merged 1 commit into
masterfrom
codex/analytic-acceleration-validation

Conversation

@KohmeiK

@KohmeiK KohmeiK commented Jul 21, 2026

Copy link
Copy Markdown
Member

Summary

This is a three-line change in one solver file:

  • finite analytic acceleration solutions near toggle points are no longer rejected solely for exceeding heuristic magnitude limits
  • link angular velocity is checked against the angular-velocity limit instead of the dimensionally incompatible joint linear-velocity limit

The kinematic equations, matrix solve, nonfinite-result check, all-zero-result behavior, public API, and every other source file remain unchanged.

Verification

  • Release library build passes
  • the existing PMKS_Verification oracle runs all five mechanisms successfully
  • all five MATLAB-PMKS comparisons pass every aligned row at the existing tolerances

@KohmeiK
KohmeiK requested review from AlexG1031 and removed request for AlexG1031 July 21, 2026 16:09
@KohmeiK
KohmeiK force-pushed the codex/analytic-acceleration-validation branch from 7413394 to 30a68e1 Compare July 21, 2026 16:27
@KohmeiK KohmeiK changed the title Validate analytic acceleration solutions by residual Allow finite accelerations near toggle points Jul 21, 2026
@KohmeiK
KohmeiK force-pushed the codex/analytic-acceleration-validation branch from 30a68e1 to 85a6ab0 Compare July 21, 2026 16:38
@KohmeiK KohmeiK changed the title Allow finite accelerations near toggle points Fix acceleration limits and link velocity guard Jul 21, 2026
@KohmeiK
KohmeiK merged commit 644b26c into master Jul 21, 2026
@KohmeiK
KohmeiK deleted the codex/analytic-acceleration-validation branch July 21, 2026 16:40
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