Solve with AdjointFactorization of Cholmod and Adjoint rhs

[dkarrasch]

Fixes #594.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.06%. Comparing base (5f52721) to head (c776ef6).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #595   +/-   ##
=======================================
  Coverage   84.06%   84.06%           
=======================================
  Files          12       12           
  Lines        9191     9192    +1     
=======================================
+ Hits         7726     7727    +1     
  Misses       1465     1465           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jonas-schulze]

Thanks for the swift solution! I wonder why the new method of (\) is not covered by the tests. 👀

Could this be backported to Julia 1.10, please?

[dkarrasch]

Could this be backported to Julia 1.10, please?

I'm not sure about that. @ViralBShah @KristofferC This was failing due to a "missing method". I guess this counts more like a "new feature" rather than "bugfix".

@jonas-schulze If we decide against backporting, you could commit type piracy in your package and define this one method in a version-dependent branch...? I know it's not optimal, but I personally have used all my credits (if I ever had some) for arguing in favor of a semver-breaking action. 😅

[ViralBShah]

I think it is ok to backport it, but @KristofferC should be the person to ask.