Skip to content

Conversation

@dkarrasch
Copy link
Member

@dkarrasch dkarrasch commented Feb 8, 2022

@dkarrasch dkarrasch added the linear algebra Linear algebra label Feb 8, 2022
@dkarrasch dkarrasch changed the title Make cholesky handle HermOrSym of structured matrices Make cholesky handle AbstractMatrix Feb 9, 2022
@dkarrasch
Copy link
Member Author

I only now realize that this is somewhat duplicate to #40690, though it includes some special handling for some special matrices, and (currently) doesn't relax the stridedness assumption on inplace cholesky!.

@dlfivefifty Do you have an example for a non-strided matrix that should cooperate with cholesky!? I could include it here if you don't mind.

@dlfivefifty
Copy link
Contributor

Banded matrices

@dkarrasch
Copy link
Member Author

Do we have some minimal testhelper thing for that, do you know?

@dkarrasch
Copy link
Member Author

I'm not sure we have an easy way to test this, but I checked that cholesky! works on a BandedMatrix, so that extension should be fine.

@dkarrasch dkarrasch merged commit 635449d into master Feb 13, 2022
@dkarrasch dkarrasch deleted the dk/cholstructured branch February 13, 2022 12:15
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request Feb 17, 2022
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linear algebra Linear algebra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cholesky fails for Hermitian{.., Diagonal}

3 participants