-
Notifications
You must be signed in to change notification settings - Fork 33
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
Algorithm and Circuit Improvements #22
Algorithm and Circuit Improvements #22
Conversation
…ng sparse vectors
…n using more than 32 qubits
♻️ unified QPE and IQPE algorithms ✅ adapted dynamic circuit tests for new improvements 🐛 fixed some bugs that prevented the QPE algorithms from working for more than 32 qubits ♻️ switched up gate order in QPE algorithm
Codecov Report
@@ Coverage Diff @@
## master #22 +/- ##
========================================
+ Coverage 80.3% 80.5% +0.2%
========================================
Files 41 41
Lines 6603 6947 +344
========================================
+ Hits 5305 5597 +292
- Misses 1298 1350 +52
Continue to review full report at Codecov.
|
…d operations to 32bits
🐛 fixed BV algorithm which was not implemented properly ✨ added dynamic circuit variant of BV algorithm ✨ added routine that automatically generated an instance for a given number of qubits ✅ added tests for new functions
⚡ DD-tailoring for diffusion operation ⚡ improve algorithm description so that it works for an arbitrary number of qubits 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: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: Lukas Burgholzer <[email protected]>
Signed-off-by: burgholzer <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* ♻️ split measurement routine into probability determination and actual measurement :sparkles: added new routine for computing the fidelity of measurement outcomes between a decision diagram and a state array
This PR primarily brings many improvements for pre-defined algorithms and the handling of dynamic circuits. In particular it:
simulate
routine with measurementsQuantumComputation
to achieve a canonical orderingDepends on cda-tum/dd_package#26.