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

Update/polish neutral atom device tutorial. #3491

Merged
merged 7 commits into from Nov 19, 2020

Conversation

rmlarose
Copy link
Contributor

Updates, formats, and removes Grover's algorithm tutorial at the end as it seems out of scope.

@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label Nov 10, 2020
@balopat balopat added the kind/docs Documentation related problems, ideas, requests label Nov 11, 2020
Copy link
Contributor

@balopat balopat left a comment

Choose a reason for hiding this comment

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

LGTM with some nits

docs/tutorials/educators/neutral_atom.ipynb Outdated Show resolved Hide resolved
docs/tutorials/educators/neutral_atom.ipynb Outdated Show resolved Hide resolved
docs/tutorials/educators/neutral_atom.ipynb Outdated Show resolved Hide resolved
docs/tutorials/educators/neutral_atom.ipynb Outdated Show resolved Hide resolved
"\n",
"Some examples of valid moments are given below. We know the moments we constructed were valid because the circuit allowed us to add them knowing that they needed to pass the neutral_device validation methods."
"# Uncommenting raises a ValueError.\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd rather put a try/except here and print out the error that "as expected we ran into an error: ...` Similar to https://cirq.readthedocs.io/en/latest/docs/protocols.html

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like this much better, thanks! Added for each example.

@balopat balopat added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Nov 19, 2020
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Nov 19, 2020
@CirqBot CirqBot merged commit 8e884b8 into quantumlib:master Nov 19, 2020
@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 Nov 19, 2020
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. kind/docs Documentation related problems, ideas, requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants