Closed mfherbst closed 3 years ago
This pull request introduces 1 alert when merging 2ab0043573a17436cbdc5271f6b25017ce21ebe4 into 96c6b9de2ad9cb9feae09d3187fe0adab864adb4 - view on LGTM.com
new alerts:
This pull request introduces 1 alert when merging ee468ba909194daf738ab4ed105ab8bb6355e0a5 into 96c6b9de2ad9cb9feae09d3187fe0adab864adb4 - view on LGTM.com
new alerts:
No worries. I hope it's doing good. I first implemented it in Julia where I really treated it super badly when it comes to numerics, then I transferred it to python and checked every step. It might still be wrong but the final residuals will always tell you, so at worst it just does not find the eigenvalues, but wrong results will not go unnoticed.
Thanks for looking at the code and documentation!
Less useful for standard ADC use cases, but very useful for debugging and for special applications (e.g. large number of eigenpairs).
TODO