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

⚡ DD Package Dynamic Memory Management #273

Closed
wants to merge 14 commits into from
Closed

Conversation

tyi1025
Copy link

@tyi1025 tyi1025 commented May 4, 2023

WIP that build on cda-tum/dd_package#158 as well as cda-tum/mqt-core#304 and aims to bring dynamic memory management to the DD package.

@tyi1025 tyi1025 requested a review from burgholzer May 4, 2023 15:34
@tyi1025 tyi1025 self-assigned this May 4, 2023
@tyi1025 tyi1025 changed the title Updated .gitmodules QCEC branch for the dd-package work May 4, 2023
@tyi1025 tyi1025 added c++ Anything related to C++ code submodules Pull requests that update Submodules code and removed c++ Anything related to C++ code labels May 4, 2023
@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Merging #273 (8468496) into main (6670228) will not change coverage.
The diff coverage is n/a.

❗ Current head 8468496 differs from pull request most recent head e6a74a5. Consider uploading reports for the commit e6a74a5 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #273   +/-   ##
=====================================
  Coverage   96.1%   96.1%           
=====================================
  Files         34      34           
  Lines       1737    1737           
  Branches     212     212           
=====================================
  Hits        1670    1670           
  Misses        67      67           
Flag Coverage Δ
cpp 95.8% <ø> (ø)
python 97.2% <ø> (ø)

see 1 file with indirect coverage changes

@burgholzer burgholzer changed the title QCEC branch for the dd-package work ⚡ DD Package Dynamic Memory Management May 4, 2023
@burgholzer burgholzer added the enhancement Anything related to improvements of the existing library label May 4, 2023
@burgholzer
Copy link
Member

Closing for now in favour of cda-tum/mqt-core#447

@burgholzer burgholzer closed this Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Anything related to improvements of the existing library submodules Pull requests that update Submodules code
Projects
Status: Done
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants