[Mass] Replace doUpdateInternal by callback: UniformMass

[hugtalbot]

In the spirit of #3900 and following #3924, this PR applies the change on the UniformMass.

To be noted, the componentState must be added to trigger the callback (done in addMDx(), addMToMatrix, accFromF and buildMassMatrix)

[ci-depends-on https://github.com/SofaDefrost/SoftRobots/pull/272] [ci-depends-on https://github.com/sofa-framework/Registration/pull/14]

---

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

• it builds with SUCCESS for all platforms on the CI.
• it does not generate new warnings.
• it does not generate new unit test failures.
• it does not generate new scene test failures.
• it does not break API compatibility.
• it is more than 1 week old (or has fast-merge label).

[hugtalbot]

only the MeshMatrixMass to go @damienmarchal and .. soon goodbye doUpdateInternal !

[hugtalbot]

trackInternalData to remove

[fredroy]

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

[alxbilger]

only the MeshMatrixMass to go @damienmarchal and .. soon goodbye doUpdateInternal !

I like your definition of "soon" 😆

[alxbilger]

Is this->trackInternalData(d_vertexMass); still necessary? @hugtalbot

[hugtalbot]

Similar feature is merged for the ConstantForceField in #3924. It adds separate callbacks for each data but a hacky flag allows to avoid data cross-dependency by de-activating all inputs non-set at the initialization. It could be done here as well but a more robust implementation (Data, callback) could be found.

Add a unit test to make sure that no data cross-dependency is triggered. Manual test as well.

[hugtalbot]

it seems from the CI that many tests and scene examples are using UniformMass without any input mass info .. :clown_face: :japanese_goblin:

[hugtalbot]

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

[hugtalbot]

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

[sofabot]

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

To unlock the merge button, you must

• Merge or close https://github.com/SofaDefrost/SoftRobots/pull/272 _For this build, SoftRobots/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/SofaDefrost/SoftRobots GITTAG 255fb151774d9480fe30bb16fb0d4bbefbd5b3c7
• Merge or close https://github.com/sofa-framework/Registration/pull/14 _For this build, Registration/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/sofa-framework/Registration GITTAG 65188decc71b7d49e792415b6f3e6cdf058744f9

[alxbilger]

@hugtalbot Could you explain why this PR triggers errors when the mass does define any parameter?

[sofabot]

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

To unlock the merge button, you must

• Merge or close https://github.com/SofaDefrost/SoftRobots/pull/272 _For this build, SoftRobots/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/SofaDefrost/SoftRobots GITTAG 255fb151774d9480fe30bb16fb0d4bbefbd5b3c7

[sofabot]

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

To unlock the merge button, you must

• Merge or close https://github.com/SofaDefrost/SoftRobots/pull/272 _For this build, SoftRobots/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/SofaDefrost/SoftRobots GITTAG 255fb151774d9480fe30bb16fb0d4bbefbd5b3c7

[sofabot]

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

To unlock the merge button, you must

• Merge or close https://github.com/SofaDefrost/SoftRobots/pull/272 _For this build, SoftRobots/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/SofaDefrost/SoftRobots GITTAG 6a73e1e775929a09bee2f4aafe49f01cdf3970be

[sofabot]

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

To unlock the merge button, you must

• Merge or close https://github.com/SofaDefrost/SoftRobots/pull/272 _For this build, SoftRobots/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/SofaDefrost/SoftRobots GITTAG 6a73e1e775929a09bee2f4aafe49f01cdf3970be

[sofabot]

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

To unlock the merge button, you must

• Merge or close https://github.com/SofaDefrost/SoftRobots/pull/272 _For this build, SoftRobots/ExternalProjectConfig.cmake.in will be edited by builders with GIT_REPOSITORY https://github.com/SofaDefrost/SoftRobots GITTAG 6a73e1e775929a09bee2f4aafe49f01cdf3970be

[sofabot]

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

All dependencies are merged/closed and all ExternalProject pointers are up-to-date. Congrats! :+1: