Syncing code-quality-improv with main
#10
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
* Update _version.py * Add IID Pauli noise model * Remove unused imports * Fix issue with linting * Add iid noise example to decoder.py Allow user to change between cv and dv noise examples. * Run black * No need to specify inner decoder for iid case * Run black on test_iid_noise * Remove uniform weight assign * Fix typos and change data function * Fix test docstring * Improve test * Add docstring to apply_noise * Update flamingpy/decoders/decoder.py * Update iid.py * Update flamingpy/noise/iid.py * Delete pauli.py * Update _version.py * Update docs; run black * Update _version.py Co-authored-by: Narimanium <25132802+nariman87@users.noreply.github.com> Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com>
ilan-tz
added a commit
that referenced
this pull request
Apr 16, 2022
* Update _version.py * Update _version.py * Pauli noise (#8) (#10) * Update _version.py * Add IID Pauli noise model * Remove unused imports * Fix issue with linting * Add iid noise example to decoder.py Allow user to change between cv and dv noise examples. * Run black * No need to specify inner decoder for iid case * Run black on test_iid_noise * Remove uniform weight assign * Fix typos and change data function * Fix test docstring * Improve test * Add docstring to apply_noise * Update flamingpy/decoders/decoder.py * Update iid.py * Update flamingpy/noise/iid.py * Delete pauli.py * Update _version.py * Update docs; run black * Update _version.py Co-authored-by: Narimanium <25132802+nariman87@users.noreply.github.com> Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> Co-authored-by: Maxime Tremblay <52462375+maxtremblay@users.noreply.github.com> Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Testing macro_reduce.py with generators * Updating macro_reduce.py * Updating macro_reduce.py * Updating macro_reduce.py * Updating .pylintrc * Update pull_request_template.md * Updating .pylintrc * Updating .pylintrc * Updating test_decoder.py * Update _version.py * Updating test_decoder.py * Running black ... * Updating test_Decoder.py * Updating test_decoder.py * Updating test_decoder.py * Updating test_decoder.py ... * Update .gitignore * Updating .pylintrc * Updating .pylintrc * Testing the removal of 'too-many-locals' from .pylintrc * Testing the removal of "redefined-builtin" in .pylintrc * Experimenting the removal of 'too-many-lines' from .pylintrc * Testing the removal of 'invalid-name' from .pylintrc * Updating .pylintrc * Updating __init__.py and macro_reduce.py * Removing trailing spaces in macro_reduce.py * Running black ... * Updating test_init.py * Added .coveragerc * Running black ... * Updating macro_reduce.py * Updating ops.py * Running black ... * Removing sparse parameter from SCZ() of ops.py * Updating ops.py ... * updating ops.py ... * Updating surface_code.py * Removing TODO items and running black ... * Updating surface_code.py * Updating decoding.py, gkp.py, and viz.py * Updating viz.py * Updating viz.py ... * Updating test_graphstates.py ... * Updates to TestCVLayer class of test_graphstates.py * Update README.md * Updating test_surface_code.py * Running black ... * Update flamingpy/codes/surface_code.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/codes/surface_code.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/codes/surface_code.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/codes/surface_code.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update .coveragerc * Create CONTRIBUTING.md * Update README.md * Update pull_request_template.md * Improve pylint score (#13) * refactor [codes] - surface_code * format - black * format - docformatter * format - docformatter * make the docformatter happy once again 🥲 * fix [github] - YAML: remove trailing spaces * refactor [ops] - initialize properties on `__init__` - add `sparse` kwarg to `SCZ_mat` * test [ops] - SCZ_mat: sparse kwarg * refactor [ops] - remove "dangerous default values" * refactor [ops] - refactor logic from __init__ * refactor [ops] - CVLayer: remove logic from __init__ * pylint [ops] - disable unneeded pylint messages * Update _version.py * Importing a copy of CHANGELOG.md from SF * Update README.md * Allowing no-self-use for test files and removing all non-pytest decorators * Update pull_request_template.md * Update contribution.rst * Update CHANGELOG.md * Update README.md * Update pull_request_template.md * Update pull_request_template.md * Update surface_code.py * Update ops.py * Update README.md * Update contribution.rst * Update README.md * Update CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Rename CODE_OF_CONDUCT.md to code_of_conduct.md * Update pull_request_template.md * Moving CONTRIBUTING.md ... * Update pull_request_template.md * Update CHANGELOG.md * Update _version.py * Fix drawing of stabilizer graph for zero syndrome (#15) * Fix drawing of stabilizer graph for zero syndrome * Ran black * Fix docstring Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Fix docstring Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Remove unused 'code' argument * Update stabilizer_graph.py * Generalize stab graph edge weights assignement * Add conversion to nx and default drawing to rx * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Add check for high low nodes before setting weight * Ran formatter * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Fix rx edge weight function * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Remove code argument for shortest path * Ran formatter * Update stabilizer_graph.py * Run black, docformatter * Update _version.py Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> Co-authored-by: Maxime Tremblay <52462375+maxtremblay@users.noreply.github.com> Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Updating unit tests ... * Updating test_benchmarks_examples.py and test_graphstates.py * [Second Try] Syncing with main (#16) * Fix drawing of stabilizer graph for zero syndrome * Ran black * Fix docstring Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Fix docstring Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Remove unused 'code' argument * Update stabilizer_graph.py * Generalize stab graph edge weights assignement * Add conversion to nx and default drawing to rx * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Add check for high low nodes before setting weight * Ran formatter * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Fix rx edge weight function * Update flamingpy/codes/graphs/stabilizer_graph.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Remove code argument for shortest path * Ran formatter * Update stabilizer_graph.py * Run black, docformatter * Update _version.py Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> Co-authored-by: Maxime Tremblay <52462375+maxtremblay@users.noreply.github.com> Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update CHANGELOG.md * Update _version.py * Update .github/CHANGELOG.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update .github/CHANGELOG.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update .github/CHANGELOG.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update flamingpy/codes/surface_code.py Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update flamingpy/codes/surface_code.py Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update .github/CHANGELOG.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update flamingpy/cv/ops.py Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update flamingpy/cv/ops.py Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update flamingpy/codes/surface_code.py Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update README.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update flamingpy/codes/surface_code.py Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update CHANGELOG.md * Update CHANGELOG.md * Update pull_request_template.md * Update README.md * Update README.md * Updating README.md and surface_code.py * Update index.rst * Update index.rst * Update contribution.rst * Replacing CONTRIBUTING.md with CONTRIBUTING.rst ... * Update contribution.rst * Update CHANGELOG.md * Update .github/CHANGELOG.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update .github/CHANGELOG.md Co-authored-by: Theodor <theodor.isacsson@gmail.com> * Update .github/CONTRIBUTING.rst Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update .github/CONTRIBUTING.rst Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update .github/pull_request_template.md Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update README.md Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update tests/unit/test_graphstates.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Update flamingpy/cv/ops.py Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> * Running black ... * Update ops.py * Update CHANGELOG.md * Update README.md * Modify SurfaceCode.generate_stabilizers slightly Co-authored-by: Maxime Tremblay <52462375+maxtremblay@users.noreply.github.com> Co-authored-by: maxtremblay <m@xtremblay.ca> Co-authored-by: ilan-tz <57886357+ilan-tz@users.noreply.github.com> Co-authored-by: Sebastián Duque Mesa <675763+sduquemesa@users.noreply.github.com> Co-authored-by: Theodor <theodor.isacsson@gmail.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.
No description provided.