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
Take-2 of polar-decomposition #1050
Take-2 of polar-decomposition #1050
Conversation
Code inspired by this thread: dimforge#656 Main person behind this is LucasCampos
Add one more test for decomposition of polar decomposition of rectangular matrix
Thanks! I took the liberty of making a few small changes:
The new proptest test is failing right now. I haven’t investigated in details yet to determine if this is a bug, or if the test are just failing because we used an epsilon that is too restrictive. |
@sebcrozet Thank you for the review and the additions. Not sure if you plan to look into it further but I will take look into the bug with the prop test and return back with a fix/solution |
Comment out unitary check for now
I zeroed in on the causes.
Fixes applied and tests pass locally |
Perfect, thanks! |
Thank you |
Code inspired by this PR/thread: #656
Main person behind this is @LucasCampos
This PR implements matrix polar decomposition