Skip to content
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

Quantum Week Tutorial Related Improvements #53

Merged
merged 4 commits into from
Sep 20, 2024
Merged

Quantum Week Tutorial Related Improvements #53

merged 4 commits into from
Sep 20, 2024

Conversation

KevinMTO
Copy link
Collaborator

@KevinMTO KevinMTO commented Sep 8, 2024

Description

This PR fixes bugs and code code usability issues along the compilation side of the stack.
New compilation methods have been added for the default selection of compilation flags.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

Copy link

codecov bot commented Sep 8, 2024

Codecov Report

Attention: Patch coverage is 35.29412% with 88 lines in your changes missing coverage. Please review.

Project coverage is 75.7%. Comparing base (590d98f) to head (e081295).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/mqt/qudits/simulation/backends/innsbruck_01.py 29.0% 39 Missing ⚠️
src/mqt/qudits/compiler/dit_compiler.py 44.7% 21 Missing ⚠️
...er/twodit/entanglement_qr/phy_ent_qr_cex_decomp.py 13.3% 13 Missing ⚠️
...er/compilation_minitools/naive_unitary_verifier.py 18.1% 9 Missing ⚠️
src/mqt/qudits/quantum_circuit/circuit.py 25.0% 6 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main     #53     +/-   ##
=======================================
- Coverage   76.4%   75.7%   -0.7%     
=======================================
  Files        117     118      +1     
  Lines       6123    6242    +119     
  Branches     279     279             
=======================================
+ Hits        4679    4731     +52     
- Misses      1444    1511     +67     
Flag Coverage Δ *Carryforward flag
cpp 63.7% <ø> (ø) Carriedforward from f62929d
python 79.9% <35.2%> (-1.0%) ⬇️

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
...dits/compiler/naive_local_resynth/local_resynth.py 95.1% <100.0%> (+0.6%) ⬆️
...g_aware_transpilation/phy_local_adaptive_decomp.py 78.0% <ø> (ø)
...er/twodit/entanglement_qr/log_ent_qr_cex_decomp.py 85.4% <100.0%> (ø)
src/mqt/qudits/simulation/backends/__init__.py 100.0% <100.0%> (ø)
src/mqt/qudits/simulation/qudit_provider.py 55.8% <100.0%> (ø)
src/mqt/qudits/quantum_circuit/circuit.py 89.3% <25.0%> (-2.4%) ⬇️
...er/compilation_minitools/naive_unitary_verifier.py 81.3% <18.1%> (-14.5%) ⬇️
...er/twodit/entanglement_qr/phy_ent_qr_cex_decomp.py 25.8% <13.3%> (-5.4%) ⬇️
src/mqt/qudits/compiler/dit_compiler.py 54.8% <44.7%> (-10.3%) ⬇️
src/mqt/qudits/simulation/backends/innsbruck_01.py 29.0% <29.0%> (ø)

... and 1 file with indirect coverage changes

@KevinMTO KevinMTO changed the title Testing Improvements Quantum Week Tutorial Related Improvements Sep 20, 2024
@KevinMTO KevinMTO added bug Something isn't working enhancement New feature or request fix usability minor python and removed bug Something isn't working labels Sep 20, 2024
@KevinMTO KevinMTO merged commit 14dd85a into main Sep 20, 2024
25 of 27 checks passed
@KevinMTO KevinMTO deleted the sw_improve branch September 20, 2024 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant