-
Notifications
You must be signed in to change notification settings - Fork 80
Add Rotation Synthesis subpackage #1749
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
Conversation
|
what's the deal with the requirements file? do you need to add mpmath? |
|
@mpharrigan I thought I did but Qualtran already depends on mpmath transitively through sympy |
|
can you go ahead and add it to |
|
@mpharrigan done |
mpharrigan
left a comment
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.
reviewed offline
The Rotation-Synthesis subpackage implements the state of the art rotation synthesis protocols for compiling$SU(2)$ unitaries into clifford+T gates. The current PR has the code for synthesis Z-rotations and I will send another PR for arbitrary axis rotations.
Please refer to
qualtran/rotation_synthesis/README.mdfor usage examples.