-
Notifications
You must be signed in to change notification settings - Fork 4
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
Adding preliminary QRE for mn_mono #8
base: main
Are you sure you want to change the base?
Conversation
…2-527/mn_mono_qre
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.
Overall this looks great! I had a couple questions and small comments. Once those are taken care of, we can merge.
with a few simplifying assumptions. First, the ground-state overlap is assumed to be | ||
0.8. Second, the double-factorized truncation threshold is assumed to be 1e-3 Ha. | ||
|
||
In order to run this script to work, you must have credentials with read access the |
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.
Small grammar edit:
In order to run this script to work,
], | ||
"compute_hardware_type": "quantum_computer", | ||
"compute_details": { | ||
"description": "Double factorized QPE resource estimates based on methodology of arXiv:2406.06335. Uses PyLIQTR logical resource estimates with BenchQ footprint analysis. Ground-state overlap assumed to be 0.8 and double-factorized truncation threshold to be 1e-3 Ha. Note that the truncation error is not included in the error bounds and that the SCF compute time is not included in the preprocessing time." |
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.
Might be good to capture the classical hardware used for the pre-processing. It's not needed at this point, but would be nice to have
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.
Do you want to remove this?
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.
Do we want this to live here or should it sit somewhere in the instances directory?
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'm agnostic to the answer here, just want to check in.
This PR adds a preliminary QRE for mn_mono as well as the python code used to generate it. This will allow us to test out the handoff to the bubble ml tool. Note that this QRE is somewhat crude as described in the compute_details section of the JSON.