-
Notifications
You must be signed in to change notification settings - Fork 9
185 official device simulations #208
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
…st skip until we generalize the observable though)
…xpectation value via sampling
📊 Benchmark Summary (ucc-benchmarks-8-core-U22.04)Comparing new 9efbf58 to base 42ca28c:
🔍 See full benchmark table
📜 See standalone benchmark tablesNew results:
Baseline results:
|
📊 Benchmark Summary (ucc-benchmarks-8-core-U22.04)Comparing new 0202486 to base 42ca28c:
🔍 See full benchmark table
📜 See standalone benchmark tablesNew results:
Baseline results:
|
📊 Benchmark Summary (ucc-benchmarks-8-core-U22.04)Comparing new 5bc192e to base 42ca28c:
🔍 See full benchmark table
📜 See standalone benchmark tablesNew results:
Baseline results:
|
📊 Benchmark Summary (ucc-benchmarks-8-core-U22.04)Comparing new 48f88bf to base 42ca28c:
🔍 See full benchmark table
📜 See standalone benchmark tablesNew results:
Baseline results:
|
|
This one ended up a smidge more complicated than anticipated:
Simulation resultsHere are a couple plots of the results, for 5 qubits ( and 7 qubits ( Pending the plot improvements in #156 to separate out different benchmark circuits and compare all combinations of noiseless/noisy/compiled/uncompiled, this is what we are seeing. At very least we see the errors are larger for the larger qubit count, as we would expect in NISQ devices. Analysis and next stepsI would expect by default, the noiseless simulations should all have abs relative errors ~0, which is true for Prep-Select and QFT, but not for QCNN and QV. (QAOA also looks wrong, but that I can chalk up this up to the fact the observable is a subset of a larger instance of the problem defined for 10 qubits). This may suggest that our observables are still of not quite right for these benchmark circuits (which may explain the roadblocks I have been hitting in #97 ).
|
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.
This is fixing the bug I have encountered multiple times in not being able to properly load local data (e.g. #129 ).
| from ..results import SimulationMetrics | ||
|
|
||
|
|
||
| def calc_computational_basis_expectation( |
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.
I think these are no longer used; they were part of debugging.
| latest_date = suite_results.metadata.uid_timestamp.strftime("%Y-%m-%d") | ||
|
|
||
| df = to_df_simulation(suite_results) | ||
| if suite_results is not None: |
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.
Messy, need to remove, this was part of debugging.
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.
Probably overkill since I don't expect we will keep using the "filtered" QAOA instance for smaller qubit counts going forward.


No description provided.