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

🚚 Move to MQT Core package #145

Merged
merged 9 commits into from
Jul 8, 2023
Merged

🚚 Move to MQT Core package #145

merged 9 commits into from
Jul 8, 2023

Conversation

burgholzer
Copy link
Member

@burgholzer burgholzer commented Jul 7, 2023

Description

This PR replaces the QFR submodule with the newly introduced MQT Core library that subsumes the QFR, DD package, and ZX package. At the moment, this entails no functional changes and only required minor adaptions.

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.

Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
@burgholzer burgholzer added dependencies Pull requests that update a dependency file submodules Pull requests that update Submodules code labels Jul 7, 2023
@burgholzer burgholzer self-assigned this Jul 7, 2023
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
Signed-off-by: Lukas Burgholzer <lukas.burgholzer@jku.at>
@github-actions
Copy link

github-actions bot commented Jul 7, 2023

Cpp-Linter Report ✔️

No problems need attention.

Have any feedback or feature suggestions? Share it here.

@codecov
Copy link

codecov bot commented Jul 8, 2023

Codecov Report

Merging #145 (633e44c) into main (af7d70d) will not change coverage.
The diff coverage is 100.0%.

@@          Coverage Diff          @@
##            main    #145   +/-   ##
=====================================
  Coverage   85.6%   85.6%           
=====================================
  Files         20      20           
  Lines       1362    1362           
  Branches     190     190           
=====================================
  Hits        1166    1166           
  Misses       196     196           
Flag Coverage Δ
cpp 79.5% <ø> (ø)
python 97.2% <100.0%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
mqt/qecc/cc_decoder/color_code.py 94.8% <100.0%> (ø)
mqt/qecc/cc_decoder/decoder.py 98.2% <100.0%> (ø)

@burgholzer burgholzer merged commit 61584a9 into main Jul 8, 2023
27 checks passed
@burgholzer burgholzer deleted the mqt-core branch July 8, 2023 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file submodules Pull requests that update Submodules code
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant