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

OrderedComplexEigenDecomposition user defined sorting #173

Closed
axkr opened this issue Dec 4, 2021 · 4 comments
Closed

OrderedComplexEigenDecomposition user defined sorting #173

axkr opened this issue Dec 4, 2021 · 4 comments

Comments

@axkr
Copy link
Contributor

axkr commented Dec 4, 2021

The fields eigenvalues and eigenvectors in ComplexEigenDecomposition didn't get sorted in OrderedComplexEigenDecomposition.
Only D and V get sorted.

How can I define my own sorting and get these fields also refreshed?

public OrderedComplexEigenDecomposition(final RealMatrix matrix, final double eigenVectorsEquality,

@maisonobe
Copy link
Contributor

Is is a bug.
I have fixed it in master branch, could you check if it works for you now?

@axkr
Copy link
Contributor Author

axkr commented Dec 5, 2021

Can you also introduce a constructor for

public OrderedComplexEigenDecomposition(final FieldMatrix<Complex> matrix, ...)` 

parameter?

@maisonobe
Copy link
Contributor

Not easily.
We don't have eigen decomposition implemented in the general case, only in the real case (which however does provide
results as complex numbers when the real matrix has complex eigenvalues).
Could you open a new issue labeled as "Feature" to ask for implementing this feature?

@axkr
Copy link
Contributor Author

axkr commented Dec 5, 2021

Done #174

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

2 participants