sofa-framework / sofa

Real-time multi-physics simulation with an emphasis on medical simulation.
https://www.sofa-framework.org
GNU Lesser General Public License v2.1
871 stars 297 forks source link

[Engine.Analyze] Fix initialization of distance type #4711

Closed alxbilger closed 1 week ago

alxbilger commented 2 weeks ago

distanceType is an OptionsGroup, and it is cannot be initialized using a single value. In this case, the wrong overload of initData was picked.


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

fredroy commented 2 weeks ago

[ci-build][with-all-tests]

epernod commented 2 weeks ago

was it triggering an error? I expect this should fix a scene somewhere?

alxbilger commented 2 weeks ago

It would fix a scene, if only there was such a scene... Currently, the constructor just crashes. There are no tests and no scenes.