Skip to content

Refactor Shor's tutorial doc to use ArithmeticGate#5665

Closed
dstrain115 wants to merge 3 commits intoquantumlib:masterfrom
dstrain115:arithmetic_gate_shors
Closed

Refactor Shor's tutorial doc to use ArithmeticGate#5665
dstrain115 wants to merge 3 commits intoquantumlib:masterfrom
dstrain115:arithmetic_gate_shors

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

  • This changes ArithmeticOperation to ArithmeticGate
    since the former is deprecated.

- This changes ArithmeticOperation to ArithmeticGate
since the former is deprecated.
@dstrain115 dstrain115 requested review from a team, cduck and vtomole as code owners July 6, 2022 18:22
@CirqBot CirqBot added the size: M 50< lines changed <250 label Jul 6, 2022
@dstrain115 dstrain115 requested a review from tanujkhattar July 6, 2022 18:22
@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

@daxfohl
Copy link
Copy Markdown
Collaborator

daxfohl commented Jul 6, 2022

Can you review #5579 instead so it doesn't create conflicts?

@dstrain115
Copy link
Copy Markdown
Collaborator Author

Sure, I will defer to #5579

@dstrain115 dstrain115 closed this Jul 6, 2022
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