-
Notifications
You must be signed in to change notification settings - Fork 14
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
✨ Analog information decoding #183
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #183 +/- ##
=======================================
- Coverage 85.6% 84.6% -1.0%
=======================================
Files 20 25 +5
Lines 1369 2020 +651
Branches 190 190
=======================================
+ Hits 1172 1710 +538
- Misses 197 310 +113
|
…o analog-information-decoding
…o analog-information-decoding
…o analog-information-decoding
src/mqt/qecc/analog_information_decoding/code_construction/sparse_code_constructor.py
Fixed
Show fixed
Hide fixed
src/mqt/qecc/analog_information_decoding/simulators/analog_tannergraph_decoding.py
Fixed
Show fixed
Hide fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some quick corrections.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just finished some small cleanups. Brought all external dependencies up to date, enabled formatting for all files and linting for a couple more files than before.
I haven't really checked the code itself, but I have sufficient trust that it is doing what it intends to do. So once CI is happy, I'll go ahead and merge 🎉
Description
This PR integrates the newest analog information decoding scripts.
Checklist: