Skip to content

[Type] Add determinant tests for 2x2 and 3x3 matrices#5796

Merged
epernod merged 1 commit intosofa-framework:masterfrom
alxbilger:determinant
Dec 1, 2025
Merged

[Type] Add determinant tests for 2x2 and 3x3 matrices#5796
epernod merged 1 commit intosofa-framework:masterfrom
alxbilger:determinant

Conversation

@alxbilger
Copy link
Contributor


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).

@alxbilger alxbilger added pr: fast merge Minor change that can be merged without waiting for the 7 review days pr: status to review To notify reviewers to review this pull-request pr: test PR adding test(s) in SOFA labels Nov 27, 2025
@epernod epernod added pr: status ready Approved a pull-request, ready to be squashed and removed pr: status to review To notify reviewers to review this pull-request labels Dec 1, 2025
@epernod epernod changed the title [Type] Add determinant tests [Type] Add determinant tests for Mat2x2 and Mat3x3 Dec 1, 2025
@epernod epernod changed the title [Type] Add determinant tests for Mat2x2 and Mat3x3 [Type] Add determinant tests for 2x2 and 3x3 matrices Dec 1, 2025
@epernod epernod merged commit 2e7d868 into sofa-framework:master Dec 1, 2025
16 of 17 checks passed
@hugtalbot hugtalbot added this to the v25.12 milestone Jan 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: fast merge Minor change that can be merged without waiting for the 7 review days pr: status ready Approved a pull-request, ready to be squashed pr: test PR adding test(s) in SOFA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants