-
Notifications
You must be signed in to change notification settings - Fork 36
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
Decodoku #288
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I moved everything to utils. This is the version I'll use for the live stream on Monday (whether it eventually gets merged or not). |
dsvandet
approved these changes
Feb 21, 2023
dsvandet
added a commit
that referenced
this pull request
Feb 21, 2023
* improve process_string for 202s * overhaul process_string * add tests for single error 202s * deprecate option not to have resets * update doc string * re-enable resets=False option, but only when 202s are not used * improve tests * Combine ARC 306 and 295 (#315) * fix issue with buffer * fix another buffer bug * fix metabuffer bug * linting * classic Python, confusing 0 and False * add option for conditional resets * tests for conditional resets * fix issue with set reordering * slight offset to round buffer * typo in doc string * use correct T in test * correct string processing for rounds_per_link>5" * refine skip backs for just_finished * link and black * correct node flagged as conjugate * add extra method for probability analysis * remove glitchy behaviour for isotropic ARCs * allow logical 1 states * allow number of rounds per 202 to be varied * handle kwargs for string2nodes * added whitespace to induce new tests * base case for rep code * test T=3 * improve weight calculation * define undefined variable * more tests for logical 1 * more tests for logical 1 * move get_error_coords to code * test get_error_coords for ARCs * make error coords not fail for 202s * improve weight update * improve coord calculations * Decodoku (#288) * add decodoku * lint, black and extra files * improve variable names * fix variable name issues" " * change import path * add visualization test * fix assertations * add test for decodoku * move to utils * black --------- Co-authored-by: Drew Vandeth <dsvandet@us.ibm.com> --------- Co-authored-by: decodoku <jwo@zurich.ibm.com> Co-authored-by: James Wootton <decodoku@gmail.com>
grace-harper
pushed a commit
that referenced
this pull request
Oct 2, 2023
* add decodoku * lint, black and extra files * improve variable names * fix variable name issues" " * change import path * add visualization test * fix assertations * add test for decodoku * move to utils * black --------- Co-authored-by: Drew Vandeth <dsvandet@us.ibm.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
The
Decodoku
class has been moved from an external repo toqiskit_qec.codes
. This implements abstract versions of qubit and qutrit codes and can present them interactively using Jupyter notebooks. The required tools using Jupyter widgets have been moved toqiskit_qec.utils.visualizations
.Details and comments
ipython
andipywidgets
have been added to the requirements.