-
Notifications
You must be signed in to change notification settings - Fork 25
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
Stage1 MVP Converter Class #266
Conversation
bargmann_dm = self._wignerdm_to_bargmanndm(wigner_dm=wigner_dm) | ||
|
||
return BargmannKet( | ||
bargmann_dm.data.A[N:, N:], bargmann_dm.data.b[N:], math.sqrt(bargmann_dm.data.c) |
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.
wigner ket has no global phase information to preserve here?
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.
Yes, we miss that information from the wignerdm to bargmanndm and it asked for extra codes to get the global phase part. Are we going to add it?
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.
if it's not too much work, yes, it would be good to have it
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.
See comments, happy to do another round after this one
Co-authored-by: Filippo Miatto <[email protected]>
Co-authored-by: Filippo Miatto <[email protected]>
Co-authored-by: Filippo Miatto <[email protected]>
Context: Add the converter class to traverse different representation classes for states.
Description of the Change: With the converter, states are able to change its representations.
Benefits: The use of representation is more flexible.
Possible Drawbacks:
Related GitHub Issues: