-
Notifications
You must be signed in to change notification settings - Fork 984
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
Add interop docs and optimizers #2884
Conversation
dstrain115
commented
Apr 2, 2020
- Move QASM from circuits doc to a new interop page
- Add JSON and quirk functionality to interop page
- Add a section on optimizers to circuits page.
- Move QASM from circuits doc to a new interop page - Add JSON and quirk functionality to interop page - Add a section on optimizers to circuits page.
Ping. Would it be possible to get a review on this PR? |
docs/circuits.ipynb
Outdated
{ | ||
"cell_type": "markdown", | ||
"metadata": { | ||
"id": "77zB_UqkAwCR", | ||
"colab_type": "text" | ||
}, | ||
"source": [ | ||
"![Circuits and Moments](https://cirq.readthedocs.io/en/latest/_static/CircuitMomentOperation.png)" | ||
"![Circuits and Moments](https://cirq.readthedocs.io/en/stable/_images/CircuitMomentOperation.png)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this image not in latest
? I thought we store all these things in static
. Why the change in this link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I think this was a merge issue of some sorts, since I changed this in an early PR.
I will also go and double-check other changes to make sure I am not squashing anything once I update the branch.
docs/circuits.ipynb
Outdated
}, | ||
"source": [ | ||
"The following call will create a circuit defined by the input QASM string:" | ||
"Optimizers will have a function `optimize_circuit()` that can be used to perform this optimization. Here is a simple example that removes moments that are empty:" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: change moments that are empty
to empty moments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
* Add interop docs and optimizers - Move QASM from circuits doc to a new interop page - Add JSON and quirk functionality to interop page - Add a section on optimizers to circuits page.