ComputationalBiomechanicsLab / opensim-creator

A UI for building OpenSim models
https://opensimcreator.com
Apache License 2.0
143 stars 17 forks source link

Fix failure to assemble a model should be treated as a failure to initialize the next model+state (maybe?) #629

Open adamkewley opened 1 year ago

adamkewley commented 1 year ago

At the moment, users can add constraints into a model that prevent the model from sucessfully assembling. This then causes problems later when the model is (e.g.) used in an FD simulation, because non-assembled models cannot be integrated.

I believe the reason that OpenSim doesn't upgrade mis-assembly to an exception is because the UI can concieveably traverse non-assembling conditions during model editing. E.g. adding a component into the model might be topologically-/data-valid but non-assemble-able until some values are tweaked.

This should be visited in later releases, though, because it does imply that non-assembling models with bad constraints are actually an error state of the model, rather than a "valid" state, because non-assembling models cannot be used in some contexts.

adamkewley commented 1 year ago

E.g.