-
Notifications
You must be signed in to change notification settings - Fork 1k
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
A Cauldron of Doc fixes #5680
A Cauldron of Doc fixes #5680
Conversation
- Change Foxtail to Sycamore since Foxtail is deprecated. - Add some clarification that instructions are for linux.
Miscellaneous doc fixes around cirq - Fix indenting where it is inappropriately detected as blockquotes - Fix references and commas at a few places. - Move some docstrings from init to class so that they show up on the API reference properly. - Fix list formatting - Fix one-line summaries that are more than one line.
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.
couple of comments but otherwise LGTM
$\sum tensor_{i_0,...,i_{r-1},j_0,...,j_{r-1}} * | ||
More specifically, this computes: | ||
|
||
$$i |
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 an extra "i"?
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.
Calibration objects instead of using this directly. | ||
"""Client for the Quantum Engine API handling protos and gRPC client. | ||
|
||
Thie is the client for the Quantum Engine API that deals with the engine protos |
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.
Thie -> This
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.
served by using the `Engine`, `EngineProgram`, `EngineJob`, `EngineProcessor`, and | ||
`Calibration` objects instead of using this directly. | ||
|
||
Args: |
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.
does this render correctly on doc site? I thought you should keep the args in the init but put a description on the class for the best redering?
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.
Changed to this style.
Miscellaneous doc fixes around cirq - Fix indenting where it is inappropriately detected as blockquotes - Fix references and commas at a few places. - Move some docstrings from init to class so that they show up on the API reference properly. - Fix list formatting - Fix one-line summaries that are more than one line.
Miscellaneous doc fixes around cirq - Fix indenting where it is inappropriately detected as blockquotes - Fix references and commas at a few places. - Move some docstrings from init to class so that they show up on the API reference properly. - Fix list formatting - Fix one-line summaries that are more than one line.
Miscellaneous doc fixes around cirq
- Fix indenting where it is inappropriately detected as blockquotes
- Fix references and commas at a few places.
- Move some docstrings from init to class so that they show up on
the API reference properly.
- Fix list formatting
- Fix one-line summaries that are more than one line.