A few tweaks to `MocoExpressionBasedParameterGoal`

opensim-org / opensim-core

SimTK OpenSim C++ libraries and command-line applications, and Java/Python wrapping.

https://opensim.stanford.edu

Apache License 2.0

800 stars 323 forks source link

A few tweaks to `MocoExpressionBasedParameterGoal` #3893

Closed nickbianco closed 2 months ago

nickbianco commented 2 months ago

Brief summary of changes

A handful of changes made to some aspects of MocoExpressionBasedParameterGoal that I noticed while using this as a reference for ExpressionBasedFunction. I've also opted to change the property variable_names to simply variables.

Testing I've completed

Ran the existing tests in testMocoGoals.cpp.

Looking for feedback on...

CHANGELOG.md (choose one)

no need to update because...tweaks to existing goal.

This change is

nickbianco commented 2 months ago

Windows failure is unrelated, merging.

Thanks @AllisonJohn!