New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simple quick fix for refactor issue #207
Conversation
Codecov Report
@@ Coverage Diff @@
## main #207 +/- ##
==========================================
+ Coverage 64.00% 64.22% +0.22%
==========================================
Files 9 9
Lines 625 629 +4
==========================================
+ Hits 400 404 +4
Misses 225 225
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Okay I'm not sure I understand how switching a test from E: nevermind it had to update I guess |
What exactly do you want KLU and UMFPACK to do on a SingularException? |
Am I good to merge this, we can solve the SingularException question in a different PR? |
Yes, I want to solve the other issue with retcodes, but we need enum retcodes first SciML/SciMLBase.jl#274 |
@ChrisRackauckas acceptable? I'll add a check option to UMFPACK for 1.9, and figure out how I want to do this in KLU. But this should suffice for now, we can always set
check = false
default later once those checks are integrated upstream.