Combine ARC 306 and 295 #315
Merged
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
…nity/qiskit-qec into 295_streamline_202s
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>
dsvandet
added a commit
that referenced
this pull request
Feb 22, 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 * Update from pass through --------- Co-authored-by: Drew Vandeth <dsvandet@us.ibm.com>
grace-harper
pushed a commit
that referenced
this pull request
Oct 2, 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 * Update from pass through --------- 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
Combine recent ARC PRs