opensim-org / opensim-core

SimTK OpenSim C++ libraries and command-line applications, and Java/Python wrapping.
https://opensim.stanford.edu
Apache License 2.0
758 stars 307 forks source link

Allow empty controls table when creating time-stepping guesses #3791

Closed nickbianco closed 1 month ago

nickbianco commented 1 month ago

Fixes the failing test in testMocoInterface.cpp on branch upgrade_ipopt.

Brief summary of changes

The subtest in testMocoInterface.cpp was failing because the OCP Moco was trying to solve was actually overconstrained. Somehow the previous Ipopt stack had no issues with this problem, but the updated stack struggles with it.

These changes allow the test to be run with no control in the model. It required a few additional changes in Moco to allow empty controls tables when creating time-stepping guesses.

Testing I've completed

Looking for feedback on...

CHANGELOG.md (choose one)


This change is Reviewable