-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Extend Eigen
to keep additional information from geevx
#38483
Conversation
(Closes #13307.) |
LGTM. Can you fix the doctest? |
I just came across this PR (Google is magic sometimes) and it is probably useful to mention in the Docs somewhere how the left eigenvectors returned this way are defined. I have only confirmed this numerically, but it is probably the same as in Numpy, i.e. for the matrix |
includes proposal for #38335
Allows to opt in evaluation of left eigenvectors and condition numbers in eigen decomposition
eigen
.Uses additional flag
unitary
to support spectral operations with the decomposition.See discussion #38335