New notebook for tutorial for codes to global rotations#155
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: de2e90054c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "import numpy as np\n", | ||
| "import matplotlib.pyplot as plt\n", | ||
| "import tsim\n", | ||
| "from tsim.utils.encoder import TransversalEncoder, SteaneEncoder" |
There was a problem hiding this comment.
In this repo's src/tsim/utils/encoder.py, the exported encoder classes are TransversalEncoder and ColorEncoder5; a repo-wide search for SteaneEncoder only finds this new notebook. Because the notebook imports SteaneEncoder here before any cells define a fallback, running the tutorial against the current package fails immediately with ImportError and none of the sweep/plot cells can execute.
Useful? React with 👍 / 👎.
1 participant
Closes #119 for unitaryHack
Hi @rafaelha This PR includes a notebook for the requested tutorial that reproduces the figure 4a of the mentioned article.
Mind that up to this point, I couldn't manage to get the 3d color code experiment working and still finding out if this is because computational constraint due to the amount of gates or because of other error, but as this is marked as optional I'm sending this PR right now while I'm certain about that, but do expect an update on this at any moment. There is also a google colab instance of this if needed for quick testing.