Open clarkmiyamoto opened 1 year ago
Ready for a review?
Ready for a review?
Yeah ready!
in
Correct. If the user doesn't input ground_spacing_back
or claw_width_back
it'll default to a uniform style.
It's overall looking good to me, I'll plan to merge
What are the issues this pull addresses (issue numbers / links)?
Slight corrections & improvements to https://github.com/qiskit-community/qiskit-metal/pull/949
Did you add tests to cover your changes (yes/no)?
Yes, updated corresponding tests for
TransmonCross
.Did you update the documentation accordingly (yes/no)?
Yes
Did you read the CONTRIBUTING document (yes/no)?
yes
Summary
The standard MIT LL candle qubit has a nonuniform claw thickness. One of the assumptions in the current
TransmonCross
is that you have uniform claw thickness, so this design isn't possible in Qiskit Metal (before this PR!). This update is to give front-end users more control of their claw designs.To illustrate what I mean here's a new potential design you can create.
And yes, it is backward compatible!
Details and comments
TransmonCross
claws.connector_type = 1