In-place version

[geoffroyleconte]

All the elements of adiag should keep their sign.

LLDL.L now allocates in order to keep the existing behaviour but I can change it to something like

get_L_factor(LLDL) = ...

if that is an issue.

[codecov]

Codecov Report

Base: 83.82% // Head: 85.99% // Increases project coverage by +2.16% 🎉

Coverage data is based on head (94da186) compared to base (beecb91).
Patch coverage: 98.64% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
+ Coverage   83.82%   85.99%   +2.16%     
==========================================
  Files           1        1              
  Lines         303      357      +54     
==========================================
+ Hits          254      307      +53     
- Misses         49       50       +1     

Impacted Files	Coverage Δ
src/LimitedLDLFactorizations.jl	85.99% <98.64%> (+2.16%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.