Skip to content

Conversation

mhucka
Copy link
Collaborator

@mhucka mhucka commented Oct 2, 2025

This updates references to master to be main.

Note: one of the CI checks is timing out. It may be a transient problem involving some GitHub service. In any case, if we apply PR #901, it should result in completion, so maybe it would be better to apply #901 first.

This updates references to `master` to be `main`.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added the size: M 50< lines changed <250 label Oct 2, 2025
@mhucka mhucka requested a review from pavoljuhas October 2, 2025 04:42
" </td>\n",
" <td>\n",
" <a target=\"_blank\" href=\"https://colab.research.google.com/github/quantumlib/qsim/blob/master/docs/tutorials/q32d14.ipynb\"><img src=\"https://quantumai.google/site-assets/images/buttons/colab_logo_1x.png\" />Run in Google Colab</a>\n",
" <a target=\\"_blank\\" href=\\"https://colab.research.google.com/github/quantumlib/qsim/blob/main/docs/tutorials/q32d14.ipynb\\"><img src=\\"https://quantumai.google/site-assets/images/buttons/colab_logo_1x.png\\" />Run in Google Colab</a>\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

The JSON syntax is broken here, please fix. As a quick check you can run
jq . docs/tutorials/q32d14.ipynb >/dev/null to verify the content is parseable.

" </td>\n",
" <td>\n",
" <a target=\"_blank\" href=\"https://colab.research.google.com/github/quantumlib/qsim/blob/master/docs/tutorials/qsimcirq.ipynb\"><img src=\"https://quantumai.google/site-assets/images/buttons/colab_logo_1x.png\" />Run in Google Colab</a>\n",
" <a target=\\"_blank\\" href=\\"https://colab.research.google.com/github/quantumlib/qsim/blob/main/docs/tutorials/qsimcirq.ipynb\\"><img src=\\"https://quantumai.google/site-assets/images/buttons/colab_logo_1x.png\\" />Run in Google Colab</a>\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Similar as above.

Copy link
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

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

Updates to the notebooks broke JSON format. Otherwise LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: M 50< lines changed <250
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants