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 educators workshop heading and contents #3464
Conversation
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 did merge the other ion device PR though - can you remove that from either this one or in a new PR?
Removed! |
Failed tests due (mostly) to not including these in rtd_docs - do we want to include these in rtd_docs @balopat?
|
Let's add them under the same heading "Educator workshop" in rtd_docs/index.rst |
Automerge cancelled: A status check is failing. |
I decided to just exclude them for now - there were too many obscure sphinx issues that I couldn't solve just now. |
Automerge cancelled: A status check is failing. |
had-to-fight-with-the-linter: conf.py seems a little bit over the top to put through formatting. |
Most notebooks are for Cirq v0.5 and need updating/polishing. Putting in now as placeholder for devsite.