Skip to content

Adds transmon calibration programs#18

Merged
loganbvh merged 7 commits intomainfrom
adds-transmon-calibration-programs
Aug 22, 2022
Merged

Adds transmon calibration programs#18
loganbvh merged 7 commits intomainfrom
adds-transmon-calibration-programs

Conversation

@qci-chou
Copy link
Member

@qci-chou qci-chou commented Jun 8, 2022

  • Adds repeated pi and pio2 pulse train calibration programs

@qci-chou qci-chou requested a review from loganbvh June 8, 2022 15:05
@loganbvh
Copy link
Member

loganbvh commented Jun 8, 2022

Hi @qci-chou - at some point Black changed the way it formats the power operator (**) in some circumstances. The main branch hadn't been updated since this happened, so the formatting in the main branch was inconsistent with the rules of the current version of Black. I updated the main branch with #19 and then reformatted your branch with a89cfe3.

The new bounds for the fit_sine guess were causing some of the tests to fail, so I updated them here c4ae360.

I also had to change the number of tune_rabi() iterations in a couple of tests to get the tests to pass - not exactly sure why, but I think everything looks good now.

If the changes I made look okay to you, feel free to merge the PR

@loganbvh loganbvh merged commit a438414 into main Aug 22, 2022
@loganbvh loganbvh deleted the adds-transmon-calibration-programs branch August 22, 2022 16:44
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