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

Adding preliminary QRE for mn_mono #8

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

max-radin
Copy link
Contributor

@max-radin max-radin commented Sep 25, 2024

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.

Copy link
Collaborator

@bellonzi bellonzi left a 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
Copy link
Collaborator

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."
Copy link
Collaborator

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

Copy link
Collaborator

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?

Copy link
Collaborator

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?

Copy link
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants