Skip to content
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

Symmetric #19

Open
cc7768 opened this issue Nov 17, 2014 · 0 comments
Open

Symmetric #19

cc7768 opened this issue Nov 17, 2014 · 0 comments

Comments

@cc7768
Copy link

cc7768 commented Nov 17, 2014

Hi. First of all, this is a great idea and an important package to have for efficiency purposes.

Just a quick question/formality --I think it should also be specified that these matrices are hermitian(symmetric) in order for the Cholesky decomposition to be meaningful (see wiki). I could just be unfamiliar with the uses of the Cholesky decomposition for positive definite nonsymmetric matrices, but wanted to mention it just in case.

Also, the majority of your cases seem like they will be covariance matrices which are symmetric so this isn't relevant in those cases, but could possibly matter in cases like this where I don't think that scale matrices for an Inverse Wishart need be symmetric.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant