-
Notifications
You must be signed in to change notification settings - Fork 188
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
Bloch-Messiah returns incorrect results #728
Labels
bug
Something isn't working
Comments
nquesada
changed the title
Bloch-Messiah returns in correct results
Bloch-Messiah returns incorrect results
Aug 11, 2022
Thanks for catching this @nquesada 😬 |
I haven't been able to find the root cause of this but it also happens for |
This was referenced Sep 6, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Before posting a bug report
Expected behavior
Consider
Actual behavior
The returned matrices
O1
andO2
in the example above are not symplectic.Reproduces how often
Always
System information
Strawberry Fields: a Python library for continuous-variable quantum circuits. Copyright 2018-2020 Xanadu Quantum Technologies Inc. Python version: 3.9.7 Platform info: Installation path: Strawberry Fields version: 0.22.0-dev Numpy version: 1.21.5 Scipy version: 1.8.0 SymPy version: 1.10 NetworkX version: 2.6.3 The Walrus version: 0.20.0-dev Blackbird version: 0.4.0 XCC version: 0.1.2 TensorFlow version: 2.6.2
Source code
No response
Tracebacks
No response
Additional information
A fixed Bloch-Messiah can be found in https:/polyquantique/thewalrus/pull/2
The text was updated successfully, but these errors were encountered: