Handle LU failure and Symmetric QR

[avik-pal]

Sorry for the bad diff, there was a bad windows code that was merged which changed all the line endings

• Fixes QR for symmetric matrices
• LU also early terminates now (not sure why we have 2 copies of solve! and do_factorization, which can be merged together with a different abstraction?)

Let's wait for SciML/NonlinearSolve.jl#414 to be finished.

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 62.71%. Comparing base (4afec5a) to head (de29257).

Files	Patch %	Lines
src/factorization.jl	66.66%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #494      +/-   ##
==========================================
- Coverage   62.85%   62.71%   -0.14%     
==========================================
  Files          29       29              
  Lines        2221     2229       +8     
==========================================
+ Hits         1396     1398       +2     
- Misses        825      831       +6     

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

[avik-pal]

Let #495 go in first

[avik-pal]

This PR will be readable after that is in