-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
More quantum circuit library refactoring #13353
Conversation
One or more of the following people are relevant to this code:
|
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.
Looks great, thanks for taking care of these! I left some minor comments below.
Pull Request Test Coverage Report for Build 11661267084Details
💛 - Coveralls |
Thanks @Cryoris, I have addressed the suggestions from the code review, and also added some missing tests, and added the functions to the API as in the other PRs. |
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.
Looks great, thank you! 🙂
Summary
As a part of circuit library refactoring #13046, more quantum circuits now have their "modern" variants,
either as gates or as synthesis methods.
This PR adds:
GraphState
->GraphStateGate
FourierChecking
->fourier_checking(...)
UnitaryOverlap
->unitary_overlap(...)
HiddenLinearFunction
->hidden_linear_function(...)
PhaseEstimation
->phase_estimation(...)
Details and comments
For now these are all in Python.