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

Physical resource estimation script #66

Merged
merged 13 commits into from
Dec 16, 2024
Merged

Physical resource estimation script #66

merged 13 commits into from
Dec 16, 2024

Conversation

max-radin
Copy link
Contributor

This PR adds a script that reads logical resource estimate solution files and generates physical resource estimate solution files in order to address #45. The generated solution files include algorithm run time, overall run time, number of physical qubits, and some details about the assumed architecture.

Most fields from the LRE solution files are retained, with a few exceptions. Importantly, the solution and solver UUIDs are replaced. The LRE solution and solver UUIDs are recorded however in the logical_resource_estimate_solution_uuid and logical_resource_estimate_solver_uuid fields of solver_details.

Note that for a given task, the script may not be able to find code distances that yield the required overall success rate. In this case the task will still be included in the new solution file, but it will not contain the fields for algorithm run time etc.

Note that the example PRE config uses a physical error rate of 1e-4, which corresponds to the "optimistic" case in arXiv:2011.03494. I chose this value because I wanted to make sure that the example config will successfully generate resource estimates for at least one of the benchmark tasks. I've tested it with planted solution 1 (14 orbitals).

@jp7745
Copy link
Member

jp7745 commented Dec 11, 2024

last night I updated the README and put some content into a separate scripts/README. please move your README updates to the scripts/README file. thanks!

@max-radin
Copy link
Contributor Author

README now fixed!

@max-radin max-radin requested a review from jp7745 December 13, 2024 04:55
@jp7745 jp7745 merged commit 1ae19cd into main Dec 16, 2024
0 of 2 checks passed
@jp7745 jp7745 deleted the max-radin/two-step-pre branch December 16, 2024 14:48
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