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

[Regression] Test linear solvers for regression #2776

Closed alxbilger closed 2 years ago

alxbilger commented 2 years ago

Add scenes to test for regression. The scenes are about linear solvers. The async solvers are not added due to the unpredicted nature of the solvers (cannot be tested for regression).

[ci-depends-on https://github.com/sofa-framework/Regression/pull/23]


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 #1.

To unlock the merge button, you must

alxbilger commented 2 years ago

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

sofabot commented 2 years ago

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

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 #3.

To unlock the merge button, you must

sofabot commented 2 years ago

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

To unlock the merge button, you must

alxbilger commented 2 years ago

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

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

fredroy commented 2 years ago

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

sofabot commented 2 years ago

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

To unlock the merge button, you must

alxbilger commented 2 years ago

2 new scenes tests are failing on windows: FEMBAR_ShewchukPCGLinearSolver_BlockJacobiPreconditioner FEMBAR_ShewchukPCGLinearSolver_PrecomputedWarpPreconditioner Any idea why? What is different form the others ones you added?

That's a good question! Especially as I am on Windows. I'll have a look. Otherwise, I'll remove those tests for the moment.

alxbilger commented 2 years ago

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

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

alxbilger commented 2 years ago

@epernod @fredroy it seems the issue was due to the files paths length

fredroy commented 2 years ago

@epernod @fredroy it seems the issue was due to the files paths length

Nice find ! 👍