Skip to content

Conversation

@augustehirth
Copy link
Collaborator

@augustehirth augustehirth commented May 26, 2022

title

@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: M 50< lines changed <250 label May 26, 2022
Copy link
Collaborator

@dabacon dabacon left a comment

Choose a reason for hiding this comment

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

A few comments

@augustehirth augustehirth marked this pull request as ready for review May 27, 2022 03:16
@augustehirth augustehirth requested review from a team, cduck and vtomole as code owners May 27, 2022 03:16
@augustehirth augustehirth requested review from dabacon and maffoo May 27, 2022 03:16
@augustehirth
Copy link
Collaborator Author

@dabacon Please re-review when you can.

Copy link
Collaborator

@dabacon dabacon left a comment

Choose a reason for hiding this comment

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

LGTM

@augustehirth augustehirth merged commit e3f7f4c into quantumlib:master Jun 8, 2022
@augustehirth augustehirth deleted the visualizing_results branch June 23, 2022 16:55
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…vements. (quantumlib#5406)

* Add "Visualizing Results" subsection and polish

* Linking

* nit

* nit 2

* Feedback changes

* Link fix
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