-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Generic LDL' factorization #203
Comments
I plan to add this when time allows. The algorithms are not demanding, but the auxiliary methods and tests will take some time. |
this would be very very useful. especially since it is one of the best ways to factorize sparse symmetric matrices. |
Unlikely to receive attention prior to 0.6. Best! |
As a clarification, the feature request is a Cholesky LDLt decomposition for abstract matrices including dense matrices? |
Yes, exactly. |
I've been hitting the problem of factorizing quasi-definite matrices several times. It is supported for sparse matrices through CHOLMOD, but not for dense. |
Can't you use |
@andreasnoack Yes, that is usually a good alternative, but it is slightly annoying to have to dispatch and handle different types of factorizations to have efficient code for sparse and dense matrices. |
is there a generic dense symmetric indefinite factorization available? (for eg BigFloat dense symmetric indefinite matrices) |
I have to admit, I never really missed this since opening the issue, because it might be useful for symmetric indefinite matrices, but doesn't really help with semidefinite matrices unlike I thought then. |
Migrated this to GenericLinearAlgebra.jl in JuliaLinearAlgebra/GenericLinearAlgebra.jl#87 |
It would be nice to have a the semidefinite Cholesky or LDL' decomposition, which allows semidefinite matrices, is more stable and avoids computation of square roots. This makes most sense as generic julia algorithm, because for BLAS floats pivoted Cholesky does the trick. I remember that pivoting was difficult for abstract matrices.
cc @andreasnoack
The text was updated successfully, but these errors were encountered: