Skip to content

Conversation

@KlausC
Copy link
Contributor

@KlausC KlausC commented Nov 18, 2020

includes proposal for JuliaLang/LinearAlgebra.jl#779

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 JuliaLang/LinearAlgebra.jl#779

@dkarrasch dkarrasch added the linear algebra Linear algebra label Nov 19, 2020
@KlausC KlausC closed this Dec 9, 2020
@KlausC KlausC reopened this Dec 9, 2020
@stevengj
Copy link
Member

stevengj commented Mar 2, 2021

(Closes JuliaLang/LinearAlgebra.jl#261.)

@vtjnash vtjnash closed this Apr 3, 2021
@vtjnash vtjnash reopened this Apr 3, 2021
@vtjnash
Copy link
Member

vtjnash commented Apr 3, 2021

LGTM. Can you fix the doctest?

@vtjnash vtjnash added the merge me PR is reviewed. Merge when all tests are passing label Apr 16, 2021
@vtjnash vtjnash merged commit 9d3a7c4 into JuliaLang:master Apr 16, 2021
@simeonschaub simeonschaub removed the merge me PR is reviewed. Merge when all tests are passing label Apr 28, 2021
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
@pgunnink
Copy link

pgunnink commented Jul 7, 2021

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 H the left eigenvectors satisfyL' H == E L', see also this discussion. So the conventional definition, such as in https://mathworld.wolfram.com/LeftEigenvector.html, does not hold.

andreasnoack added a commit that referenced this pull request Jul 14, 2021
DilumAluthge pushed a commit that referenced this pull request Jul 15, 2021
#41578)

* Revert "Extend `Eigen` to keep additional information from `geevx` (#38483)"

This reverts commit 9d3a7c4.

* Remove left eigenvector handling in Float16 eigen methods.
andreasnoack added a commit that referenced this pull request Jul 19, 2021
KristofferC pushed a commit that referenced this pull request Jul 20, 2021
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.

6 participants