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

[SofaMiscCollision] Extract OBB/Capsule code into its own plugin #2820

Closed fredroy closed 2 years ago

fredroy commented 2 years ago

based on:

All OBB/Capsule code-related into its own plugin. Has kind of the same file structure as Sofa.Component.Collision.

Tests were adapted as well.

[ci-depends-on https://github.com/sofa-framework/THMPGSpatialHashing/pull/1]


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

sofabot commented 2 years ago

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

To unlock the merge button, you must

fredroy commented 2 years ago

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

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

guparan commented 2 years ago

Plugin to be externalized in the future?

hugtalbot commented 2 years ago

The plugin should then have its own repository. In this PR or another one, up to you @fredroy → wait for #2813 then ready

sofabot commented 2 years ago

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

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

hugtalbot commented 2 years ago

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

sofabot commented 2 years ago

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

To unlock the merge button, you must