-
Notifications
You must be signed in to change notification settings - Fork 615
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
[labs.dla 6] Variational KAK decomposition #6446
Conversation
Test example still fails for n=4 🫠 cc @dwierichs edit: works now 🥳 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, looking forward for this milestone to be merged! 💯
I think I looked at this code far too many times by now :D
I mostly had quite small comments/suggestions, like trying to avoid mixing "khk" and "kak", and things like that. Otherwise LGTM and happy to approve after!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll look more into the code next week, but this seems pretty good!
…o variational_khk
Co-authored-by: David Wierichs <[email protected]> Co-authored-by: Pietropaolo Frisoni <[email protected]>
…ylane into variational_khk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work @Qottmann 💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a few minor suggestions, but no blockers from me
Co-authored-by: Pietropaolo Frisoni <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Variational KHK decomposition using the adjoint representation
Based off https://arxiv.org/pdf/2104.00728 but with some important modifications, in particular using the adjoint representation which is faster and enables working with arbitrary bases
TODO
[sc-74997]