Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Ostrich_PeerJ2015.osim cannot equilibriate muscles with latest OpenSim? #765

Open
adamkewley opened this issue Sep 7, 2023 · 7 comments

Comments

@adamkewley
Copy link
Collaborator

adamkewley commented Sep 7, 2023

Repro

Loading an Ostrich model:

Now appears to throw a muscle equilibriation error:

image]

OSC Error Investigation

OSC Version OpenSim Version State
0.5.x 4.4.1 BROKEN
0.4.1 4.4 BROKEN
0.3.2 4.4 BROKEN
0.2.0 4.3 BROKEN
0.1.6 4.3 BROKEN
0.1.0 4.2 BROKEN
0.0.2 4.1 WORKING

Comparison with OpenSim GUI

  • Loads fine in OpenSim GUI
  • But OpenSim GUI will subsequently blow up with the same error if the model is simulated
  • The difference in behavior is because OSC always realizes a model's initial state to Report (so that outputs are always available for watching/plotting), whereas OpenSim GUI keeps the model in an earlier state until the user presses the simulate button
  • The crash also happens with OpenSim 4.1. So it's likely that the reason that OSC 0.2.0 (OSMV) does not crash is because earlier versions of OSC behaved the same as OpenSim GUI (i.e. only using a basic, non-realized State, because early OSC did not support output monitoring).

Conclusions

Likely to close this one because it seems that the model is broken in OpenSim 4.0+ independently of OSC. The model file itself will need to be updated/fixed for OpenSim4, rather than OSC/opensim-core requiring code modifications (unless we can prove it's a non-issue and can be fixed automatically in-code).

@adamkewley
Copy link
Collaborator Author

Also broken in 0.4.1

@adamkewley
Copy link
Collaborator Author

Also broken in 0.3.2

@adamkewley
Copy link
Collaborator Author

Also broken in 0.2.0

@adamkewley
Copy link
Collaborator Author

Also broken in 0.1.6

@adamkewley
Copy link
Collaborator Author

Also broken in 0.1.0

@adamkewley
Copy link
Collaborator Author

Not broken in osmv (0.0.2)

@adamkewley
Copy link
Collaborator Author

Dropping this issue from OSC's 0.5.2 release: it's been broken for a couple of years.

The solution to this is to upstream that opensim-core's test suite loads+realizes a collection of published models, so that developers can spot potential model breakages as changes happen.

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

No branches or pull requests

1 participant