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

[All] Propose a new gravity API #2988

Closed hugtalbot closed 11 months ago

hugtalbot commented 1 year ago

This pull request aims at making the gravity clearly appearing in object nodes as a GravityForceField. Old way to define gravity still works and is not deprecated.

Main changes:

The GravityForceField can:

Fix: #2683

[ci-depends-on https://github.com/sofa-framework/SofaPython3/pull/305] [ci-depends-on https://github.com/sofa-framework/BeamAdapter/pull/56] [ci-depends-on https://github.com/sofa-framework/ci/pull/18]


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

hugtalbot commented 1 year ago

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

hugtalbot commented 1 year ago

TODO:

One issue remains in my tests :

hugtalbot commented 1 year ago

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

sofabot commented 1 year ago

[ci-depends-on] detected during build #15.

To unlock the merge button, you must

hugtalbot commented 1 year ago

CI fails due to ExternalBehaviorModel plugin, depending on SofaGraphComponent (InteractingBehaviorModel class) itself deprecated ...

Question: isn't it time to deprecate this plugin gleichzeitig?

sofabot commented 1 year ago

[ci-depends-on] detected during build #16.

To unlock the merge button, you must

sofabot commented 1 year ago

[ci-depends-on] detected during build #17.

To unlock the merge button, you must

hugtalbot commented 1 year ago

What was written above (hidden by GH):


TODO:

One issue remains in my tests :

sofabot commented 1 year ago

[ci-depends-on] detected during build #18.

To unlock the merge button, you must

hugtalbot commented 1 year ago

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

sofabot commented 1 year ago

[ci-depends-on] detected during build #19.

To unlock the merge button, you must

sofabot commented 1 year ago

[ci-depends-on] detected during build #20.

To unlock the merge button, you must

hugtalbot commented 1 year ago

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

sofabot commented 1 year ago

[ci-depends-on] detected during build #21.

To unlock the merge button, you must

fredroy commented 1 year ago

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

sofabot commented 1 year ago

[ci-depends-on] detected during build #22.

To unlock the merge button, you must

sofabot commented 1 year ago

[ci-depends-on] detected during build #23.

To unlock the merge button, you must

sofabot commented 1 year ago

[ci-depends-on] detected during build #24.

To unlock the merge button, you must

sofabot commented 1 year ago

[ci-depends-on] detected during build #25.

To unlock the merge button, you must

hugtalbot commented 1 year ago

Not enough convincing arguments to bring such a breaking feature into SOFA Let's discuss and possibly close it at next STC#15