sofa icon indicating copy to clipboard operation
sofa copied to clipboard

[constraint.lagrangian] Create data links in ConstraintCorrection for linear solver

Open hugtalbot opened this issue 2 years ago • 2 comments

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

hugtalbot avatar Jul 21 '22 09:07 hugtalbot

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.

damienmarchal avatar Jul 21 '22 13:07 damienmarchal

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

hugtalbot avatar Jul 21 '22 19:07 hugtalbot