Skip to content

Conversation

@dabacon
Copy link
Collaborator

@dabacon dabacon commented May 26, 2022

  • Remove deprecated SingleQubitGate
  • Show Sycamore chip, not Bristlecone.
  • Fix some warnings that come up from show being called in plot methods.

This notebook did not have its execution cleared, not sure if there is a reason for this?

@dabacon dabacon requested review from a team, cduck and vtomole as code owners May 26, 2022 20:05
@dabacon dabacon requested a review from maffoo May 26, 2022 20:05
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@CirqBot CirqBot added the size: L 250< lines changed <1000 label May 26, 2022
Copy link
Collaborator

@dstrain115 dstrain115 left a comment

Choose a reason for hiding this comment

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

I would clear outputs. I don't see any reason not to.

Also, does this page still need "--pre". If not, can we get rid of that part at the top?

@dabacon dabacon requested a review from dstrain115 June 2, 2022 20:11
@dabacon
Copy link
Collaborator Author

dabacon commented Jun 2, 2022

Got rid of --pre and confirmed it works without it.

@dabacon
Copy link
Collaborator Author

dabacon commented Jun 27, 2022

This has gone stale. closing

@dabacon dabacon closed this Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs size: L 250< lines changed <1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants