Skip to content

Remove deprecated SingleQubitGate#5686

Merged
CirqBot merged 11 commits intoquantumlib:masterfrom
dstrain115:deprecate_single_qubit_gate
Jul 8, 2022
Merged

Remove deprecated SingleQubitGate#5686
CirqBot merged 11 commits intoquantumlib:masterfrom
dstrain115:deprecate_single_qubit_gate

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

Note: cirq.testing.SingleQubitGate still exists for brevity when writing tests.)

@dstrain115 dstrain115 requested review from a team, cduck and vtomole as code owners July 8, 2022 16:48
@dstrain115 dstrain115 requested a review from tanujkhattar July 8, 2022 16:48
@CirqBot CirqBot added the size: M 50< lines changed <250 label Jul 8, 2022
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@pavoljuhas
Copy link
Copy Markdown
Collaborator

One last leftover reference at

"An alternative to inheriting from `cirq.Gate` is to inherit from `cirq.SingleQubitGate`, in which case defining `_num_qubits_` is unnecessary. An example of a defining a two-qubit gate is shown below."

Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

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

LGTM after removing SingleQubitGate reference at

"An alternative to inheriting from `cirq.Gate` is to inherit from `cirq.SingleQubitGate`, in which case defining `_num_qubits_` is unnecessary. An example of a defining a two-qubit gate is shown below."

@dstrain115 dstrain115 added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Jul 8, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jul 8, 2022
@CirqBot CirqBot merged commit b47cee8 into quantumlib:master Jul 8, 2022
@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 Jul 8, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Note: cirq.testing.SingleQubitGate still exists for brevity when writing tests.)
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
Note: cirq.testing.SingleQubitGate still exists for brevity when writing tests.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: M 50< lines changed <250

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants