sofa
sofa copied to clipboard
[constraint.lagrangian] Create data links in ConstraintCorrection for linear solver
The data strings are replaced with Links! Finally!
It's obviously breaking but there is one important point: both ConstraintCorrection had a vector of LinearSolver* while in fact only one was always used. It is now back only to a single pointer.
Let's take time to discuss it with others!
@ziqiu-zeng
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).
Hello @hugtalbot,
Thanks for this PR. Replacing this string by a real link was really needed move. In your comment you are saying that from the solvers list only one was used. From what I can see there are plently of "for" loops in the code that do the same operation on each solver so can you explain a bit more why they are not needed anymore ?
EDIT: the answer was given by Hugo in gitter.
[ci-build][with-all-tests]