-
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
Fix qft-plugins for custom 'qft' gates (backport #13181) #13189
Conversation
Cherry-pick of 2245a4f has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
Thank you for opening a new pull request. Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone. One or more of the following people are relevant to this code:
|
(cherry picked from commit 2245a4f)
384833a
to
24289c9
Compare
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.
Assuming this passes lint/tests, it should now be a clean cherry-pick.
Pull Request Test Coverage Report for Build 10945733297Details
💛 - Coveralls |
(cherry picked from commit 2245a4f) Co-authored-by: Jake Lishman <[email protected]>
Summary
The
HighLevelSynthesis
transpiler pass no longer raises an exception when encountering a custom gate that is called "qft" but is not an instance of classQFTGate
. Instead, the synthesis plugins for QFT gates ignore such a gate, and the gate's definition is used (if provided).Fixes #13120 and #13174.
The PR is very short and has backport potential (if desired).
This is an automatic backport of pull request #13181 done by [Mergify](https://mergify.com).