Skip to content

🔥 Remove Teleportation gate #882

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

Merged
merged 1 commit into from
Mar 23, 2025
Merged

Conversation

burgholzer
Copy link
Member

Description

This PR removes the Teleportation gate from mqt-core in preparation for the v3 launch.
It was only ever used in one routing pass in MQT QMAP that is being phased out and was hardly documented.
Might as well remove it here. Removed code is debugged code 🤷🏼

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 added refactor Anything related to code refactoring Core Anything related to the Core library and IR c++ Anything related to C++ code labels Mar 23, 2025
@burgholzer burgholzer self-assigned this Mar 23, 2025
@burgholzer burgholzer added this to MQT Core and MQT Mar 23, 2025
@github-project-automation github-project-automation bot moved this to In Progress in MQT Core Mar 23, 2025
@github-project-automation github-project-automation bot moved this to In Progress in MQT Mar 23, 2025
@burgholzer burgholzer enabled auto-merge (squash) March 23, 2025 11:39
Copy link

codecov bot commented Mar 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.6%. Comparing base (c5b8e11) to head (838fbff).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #882   +/-   ##
=====================================
  Coverage   92.5%   92.6%           
=====================================
  Files        145     145           
  Lines      13841   13822   -19     
  Branches    2107    2107           
=====================================
- Hits       12810   12801    -9     
+ Misses      1031    1021   -10     
Flag Coverage Δ
cpp 92.3% <100.0%> (+<0.1%) ⬆️
python 98.4% <ø> (ø)
Files with missing lines Coverage Δ
include/mqt-core/ir/operations/OpType.inc 100.0% <100.0%> (ø)
...clude/mqt-core/ir/operations/StandardOperation.hpp 100.0% <ø> (ø)
src/ir/QuantumComputation.cpp 94.1% <ø> (-0.1%) ⬇️
src/ir/operations/OpType.cpp 31.8% <ø> (+0.8%) ⬆️
src/ir/operations/StandardOperation.cpp 94.6% <ø> (+<0.1%) ⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@burgholzer burgholzer added this to the MQT Core milestone Mar 23, 2025
@burgholzer burgholzer merged commit 0e993c2 into main Mar 23, 2025
31 checks passed
@burgholzer burgholzer deleted the remove-teleportation-gate branch March 23, 2025 12:04
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT Mar 23, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT Core Mar 23, 2025
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant