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

create alternative version for eigen values/vectors calculation #3016

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

bartekleon
Copy link
Contributor

solution for #3014

This is a rough version. It might not work for all cases.

@bartekleon
Copy link
Contributor Author

Checklist for implementation (unless decided otherwise):

  • !! Ensure eigenvectors are correct, not only eigenvalues
  • Ensure BigInt working
  • Ensure BigNumber working
  • Ensure Complex working

@gwhitney
Copy link
Collaborator

gwhitney commented Sep 2, 2023

There are a lot of numerical issues in computing eigenvalues and eigenvectors. The current implementation at least attempts to handle those by "balancing" the matrix, etc. I doubt the solution is to start afresh with a totally new implementation (that will likely have its own collection of shortcomings). But let's keep this concept open until the issue in #2879 is resolved.

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

Successfully merging this pull request may close these issues.

2 participants