-
Notifications
You must be signed in to change notification settings - Fork 19
🎨 MQT Bench Update: New device structure and device, code refactoring, and further improvements #429
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
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #429 +/- ##
=======================================
- Coverage 92.8% 92.4% -0.4%
=======================================
Files 43 35 -8
Lines 1969 1591 -378
=======================================
- Hits 1828 1471 -357
+ Misses 141 120 -21
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This is the first PR in a series that splits up #429 into smaller chunks that are easier to digest. Hopefully, this gets the ball rolling and gets us closer to an actual next major release. This PR removes the `benchviewer` and `evaluation` modules. Signed-off-by: burgholzer <[email protected]> Co-authored-by: Nils Quetschlich <[email protected]>
30f7f2f
to
107eb4b
Compare
I just rebased this branch on the current main and cleaned up the history a little bit. |
58c4b77
to
84201e4
Compare
…e, and qiskit_algorithm dependencies
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
84201e4
to
02238ad
Compare
Extracted from #429: 9. adds the Bernstein-Vazarani algorithm
Extracted from #429: 2. removes the placement option for the mapped TKET quantum circuits
This PR
Provider
class (resolves Remove theProvider
concept and replace it with base classes ofDevice
s #387)placement
option for the mapped TKET quantum circuitsibmq_torino
(and, with that, resolves Add 433-Qubit IBM Device with new native gate set #220 kind of)Device
class structurerandom
circuit (resolves Renamerandom
benchmark because of shadowing python's module with the same name #368)Bernstein-Vazarani
algorithmqiskit_algorithms
,qiskit_optimization
, andqiskit_nature
(making 🔥Remove most of the qiskit application module benchmarks #404 obsolete and resolves ✨ Removal of deprecated dependencies #390 as well as resolves ✨ Improving documentation to make it more explicit which benchmarks trigger an optimization procedure when generated #388)As a consequence of this PR, the
get_benchmark
function parameter name changed since it does not need the provider anymore but a gateset. However, this allows one to easily generate benchmarks for custom gatesets.This PR comes with multiple breaking changes.