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

🚸 Allow creating circuits from compound operations #690

Merged
merged 6 commits into from
Sep 12, 2024

Conversation

burgholzer
Copy link
Member

Description

This PR adds functionality to construct circuits from compound operations.
In the process, it also improves the QASM dumping functionality by making it const.
Last, but not least, it refactors the corresponding IO tests to use more convenient interfaces that are now available.

Fixes #600

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.

@burgholzer burgholzer self-assigned this Sep 12, 2024
@burgholzer burgholzer added usability Anything related to usability refactor Anything related to code refactoring Core Anything related to the Core library and IR c++ Anything related to C++ code labels Sep 12, 2024
Copy link

codecov bot commented Sep 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.1%. Comparing base (bcb8371) to head (ace3312).
Report is 104 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #690   +/-   ##
=====================================
  Coverage   92.1%   92.1%           
=====================================
  Files        125     125           
  Lines      13780   13790   +10     
  Branches    2153    2154    +1     
=====================================
+ Hits       12692   12704   +12     
+ Misses      1088    1086    -2     
Flag Coverage Δ
cpp 91.8% <100.0%> (+<0.1%) ⬆️
python 99.7% <ø> (ø)
Files with missing lines Coverage Δ
include/mqt-core/ir/QuantumComputation.hpp 91.9% <100.0%> (-0.6%) ⬇️
src/ir/Permutation.cpp 100.0% <100.0%> (ø)
src/ir/QuantumComputation.cpp 85.0% <100.0%> (+0.6%) ⬆️

@burgholzer burgholzer force-pushed the qc-from-compound-operation branch from f5dcf5e to ace3312 Compare September 12, 2024 13:51
@burgholzer burgholzer merged commit cd2e767 into main Sep 12, 2024
33 checks passed
@burgholzer burgholzer deleted the qc-from-compound-operation branch September 12, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Anything related to C++ code Core Anything related to the Core library and IR refactor Anything related to code refactoring usability Anything related to usability
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

✨ Create QuantumComputation from CompoundOperation
1 participant