-
Notifications
You must be signed in to change notification settings - Fork 989
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
Adjust docs for Getting started guide #3307
Conversation
- Minor issues found through walk-through - Google colabs are meant to be downloaded, so image links need to be absolute - Rearrange text so that project creation/selection happens before trying to enable the API - Add link to reservation colab on getting started guide.
docs/tutorials/google/colab.ipynb
Outdated
@@ -35,7 +35,7 @@ | |||
"You can retrieve ipython notebooks in the cirq repository by\n", | |||
"going to the [doc directory](https://github.com/quantumlib/Cirq/tree/master/docs). For instance, this colab template can be found [here](https://github.com/quantumlib/Cirq/blob/master/docs/tutorials/google/colab.ipynb). Select the file that you would like to download and then click the \"Raw\" button in the upper right part of the window:\n", | |||
"\n", | |||
"![Raw button](../../images/colab_github.png)\n", | |||
"![Raw button](https://cirq.readthedocs.io/en/latest/_images/colab_github.png)\n", |
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.
Why are we referring to the RTD URL when this will end up on our devsite?
Are you running this locally? Or in Colab? Where does the relative image not work?
if we absolutely need absolute links (pun intended) we could link to the raw image on github on master (a particular commit would be better) - but referring to the Readthedocs URL is definitely odd.
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.
The relative links won't work if you download the colab and then run it through colab, which you may want to do for the getting started guide and the colab template. I changed it to github absolute URLs.
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.
LGTM.
I'm not happy with this image URL workflow yet, opened an issue to tackle it #3308.
so image links need to be absolute
happens before trying to enable the API