opensim-org / opensim-moco

Solve optimal control problems for musculoskeletal models using OpenSim and direct collocation.
https://opensim.stanford.edu/moco
Apache License 2.0
58 stars 16 forks source link

MocoPeriodicityGoal bug fix for control variables #609

Closed nickbianco closed 4 years ago

nickbianco commented 4 years ago

Fixes issue #528

Brief summary of changes

Fixed a bug where the control indices that define periodicity constraints wasn't being used at all. Tried to alter the test so it failed on the original code and passed with the fix.

CHANGELOG.md (choose one)


This change is Reviewable

nickbianco commented 4 years ago

Thanks guys!