Add HeronR2 device profile#25
Closed
LakshikkaNathan wants to merge 1 commit into
Closed
Conversation
Member
|
Hi @LakshikkaNathan, thanks for your submission to #15 for unitaryHACK! 👋 We've done an initial read-through and it looks like a real, well-formed contribution. We're pulling it down to run the test suite locally tonight and will follow up with detailed feedback in the morning. Appreciate your patience! |
Member
|
We have many quality submissions and it was a tough decision, but there is still so much work to do on this package and we encourage everyone who made a submission to continue contributing and making suggestions, PRs, and issues on this repo. Thank you all! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #15
Summary
Adds
HeronR2()tosrc/profiles.jl, mirroringHeronR3(). Covers the 156-qubit heavy-hex Heron r2 processor (ibm_marrakesh, ibm_fez, ibm_kingston - one shared profile) and models the first 8 qubits as a linear chain.Values:
His a synthesized estimate (SX·RZ), not native.Testing
Exports
HeronR2and adds a construction test plus the issue-requestedcompile_blockfidelity test, leaving the existingHeronR3 constructiontest in place. The compile test runs at reduced size (n_levels=2,N_knots=5,max_iter=2) for speed. Full suite passes locally: 113 tests, Julia 1.12.6, Piccolo 1.18.Note
Frequency, anharmonicity, and coupling strength are representative values. The ibm_marrakesh calibration export lists T1/T2, gate errors and durations, readout, and CZ/RZZ errors but not these three, and backend.qubit_properties() returns frequency and anharmonicity as None. So I used representative values, matching how
HeronR3is built.AI usage
I used Claude as a coding assistant to help draft this change. I pulled the calibration myself, ran the tests, and reviewed the full patch before submitting.