Complex (Hermitian) eigenvalues/eigenvectors#105
Open
imantdaunhawer wants to merge 4 commits intojblas-project:masterfrom
Open
Complex (Hermitian) eigenvalues/eigenvectors#105imantdaunhawer wants to merge 4 commits intojblas-project:masterfrom
imantdaunhawer wants to merge 4 commits intojblas-project:masterfrom
Conversation
eigenvalues/vectors of complex matrices for the existing routine CGEEV. The implementation is ment to be consistent with the eigenvalue/vector computation for general matrices, by overloading the existing methods.
…trix (Fortran routines CHEEV and ZHEEV). Implementation of native methods, convenience methods, and tests.
…trix (Fortran routines CHEEV and ZHEEV). Implementation of native methods, convenience methods, and tests.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
on request of a colleague I've added the computation of eigenvectors/eigenvalues of complex and complex hermitian matrices as native and convenience methods. For this I received some funding from the University of Konstanz (Germany). Now, I would like to share this work with you, to uncover potential mistakes and to make a small contribution to your project.
For the computation of eigenvalues/eigenvectors of complex matrices (Fortran routines CGEEV and ZGEEV) the native methods already existed, but I added convenience methods for calling them from the
Eigenclass. For this I overloaded the existing methodsEigen.eigenvalueandEigen.eigenvectorin a way that seems consistent with the current design. Further I've implemented basic tests toTestEigen.javato check my implementation.For the computation of eigenvalues/eigenvectors of complex Hermitian matrices (Fortran routines CHEEV and ZHEEV) I added the native methods (through an addition to the Makefile which automatically generates the C code). Further, I created new convenience methods
Eigen.hermitianEigenvaluesandEigen.hermitianEigenvectorsfor which a call assumes a Hermitian input matrix (otherwise, the results might be incorrect). Again, I've included tests that cover some basic cases.I'll be glad to receive feedback on this implementation and I even have a few days left to work on further functionality related to complex and Hermitian matrices (please let me know if there are any preferences from your side).