Skip to content
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

Added r(theta, phi) gate #4455

Merged
merged 4 commits into from Aug 24, 2021
Merged

Added r(theta, phi) gate #4455

merged 4 commits into from Aug 24, 2021

Conversation

Zshan0
Copy link
Contributor

@Zshan0 Zshan0 commented Aug 21, 2021

As requested in #3415.

Made changes in _parser.py and added r gate under. I have replicated the format of u3 and set params to QasmUGate according to r(theta, phi) = u3(theta, phi - pi/2, -phi + pi/2).

Also wrote a test for the same.

@Zshan0 Zshan0 requested review from cduck, vtomole and a team as code owners August 21, 2021 19:26
@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label Aug 21, 2021
@CirqBot CirqBot added the size: S 10< lines changed <50 label Aug 21, 2021
@google-cla
Copy link

google-cla bot commented Aug 23, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no and removed cla: yes Makes googlebot stop complaining. labels Aug 23, 2021
@banrovegrie
Copy link
Contributor

@googlebot I consent

@google-cla google-cla bot added cla: yes Makes googlebot stop complaining. and removed cla: no labels Aug 23, 2021
Copy link
Collaborator

@MichaelBroughton MichaelBroughton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MichaelBroughton MichaelBroughton added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Aug 24, 2021
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Aug 24, 2021
@CirqBot CirqBot merged commit 932f065 into quantumlib:master Aug 24, 2021
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Aug 24, 2021
@banrovegrie banrovegrie deleted the Dev branch August 25, 2021 17:27
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
As requested in quantumlib#3415.

Made changes in `_parser.py` and added `r` gate under. I have replicated the format of `u3` and set params to `QasmUGate` according to `r(theta, phi) = u3(theta, phi - pi/2, -phi + pi/2)`. 

Also wrote a test for the same.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Makes googlebot stop complaining. size: S 10< lines changed <50
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants