cda-tum / mqt-qecc

MQT QECC - Tools for Quantum Error Correcting Codes
MIT License
100 stars 13 forks source link

bye ldpc #271

Closed lucasberent closed 2 weeks ago

lucasberent commented 2 weeks ago

Description

Replace the LDPC C++ dependency with direct integration of the few features needed.

Also fixes a large bunch of new clang-tidy warnings and switches to reusable MQT workflows for linting and CodeQL.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 96.62921% with 3 lines in your changes missing coverage. Please review.

Project coverage is 83.0%. Comparing base (28075c7) to head (71c4017). Report is 29 commits behind head on main.

Files with missing lines Patch % Lines
include/Code.hpp 50.0% 2 Missing :warning:
src/GF2.cpp 98.3% 1 Missing :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271/graphs/tree.svg?width=650&height=150&src=pr&token=OAw9kgumkt&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum)](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum) ```diff @@ Coverage Diff @@ ## main #271 +/- ## ======================================= + Coverage 82.7% 83.0% +0.2% ======================================= Files 46 48 +2 Lines 3978 4040 +62 Branches 353 372 +19 ======================================= + Hits 3292 3355 +63 + Misses 686 685 -1 ``` | [Flag](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum) | Coverage Δ | | |---|---|---| | [cpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum) | `84.0% <96.6%> (+0.6%)` | :arrow_up: | | [python](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum) | `82.3% <ø> (ø)` | | | [Files with missing lines](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum) | Coverage Δ | | |---|---|---| | [include/Codes.hpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=include%2FCodes.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-aW5jbHVkZS9Db2Rlcy5ocHA=) | `100.0% <ø> (ø)` | | | [include/Decoder.hpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=include%2FDecoder.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-aW5jbHVkZS9EZWNvZGVyLmhwcA==) | `44.0% <ø> (ø)` | | | [include/DecodingRunInformation.hpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=include%2FDecodingRunInformation.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-aW5jbHVkZS9EZWNvZGluZ1J1bkluZm9ybWF0aW9uLmhwcA==) | `68.4% <100.0%> (ø)` | | | [include/GF2.hpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=include%2FGF2.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-aW5jbHVkZS9HRjIuaHBw) | `100.0% <100.0%> (ø)` | | | [include/QeccException.hpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=include%2FQeccException.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-aW5jbHVkZS9RZWNjRXhjZXB0aW9uLmhwcA==) | `20.0% <ø> (ø)` | | | [include/TreeNode.hpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=include%2FTreeNode.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-aW5jbHVkZS9UcmVlTm9kZS5ocHA=) | `96.1% <ø> (ø)` | | | [src/UFDecoder.cpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=src%2FUFDecoder.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-c3JjL1VGRGVjb2Rlci5jcHA=) | `63.7% <100.0%> (-0.7%)` | :arrow_down: | | [src/UFHeuristic.cpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=src%2FUFHeuristic.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-c3JjL1VGSGV1cmlzdGljLmNwcA==) | `81.0% <ø> (ø)` | | | [src/Utils.cpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=src%2FUtils.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-c3JjL1V0aWxzLmNwcA==) | `87.0% <100.0%> (-0.2%)` | :arrow_down: | | [src/GF2.cpp](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree&filepath=src%2FGF2.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum#diff-c3JjL0dGMi5jcHA=) | `98.3% <98.3%> (ø)` | | | ... and [1 more](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum) | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/cda-tum/mqt-qecc/pull/271/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cda-tum)