-
Notifications
You must be signed in to change notification settings - Fork 55
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
\ does not work for sparse matrices with Float32 elements #111
Comments
I think the core problem is that you cannot change between float32 and float64 support in suitesparse during runtime. Edit: Actually JuliaLang/julia#14076 (comment) says that it's not even supported. |
It's not supported in at least UMFPACK and KLU. I think there's promotions to Float64 somewhere, they just may not be available in the |
The issue with the given problem is that the sparse matrix is recognized as Hermitian, so it gets wrapped as It should perhaps be widened to include all those nzval-types that get promoted to Float64 if necessary. |
Note that CHOLMOD does not support Float32, but UMFPACK does (and Julia should probably as well). We do emit the warning for CHOLMOD
So what we need to do is what @dkarrasch suggests above. |
As discussed in https://stackoverflow.com/questions/70648351/getting-an-error-when-using-backslash-operator-on-sparse-matrix-and-dense-vector/70662601#70662601,
\
does not work for sparse matrices with Float32 elements.It would be nice to bring back Float32 support in sparse solvers. The issue is that
\
callslu!
, which does not have a matching method. Naturallylu!
should not promote, but\
should check and promote.The text was updated successfully, but these errors were encountered: