-
Notifications
You must be signed in to change notification settings - Fork 6
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
Solve missing plot for protocols involving chi2 #744
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #744 +/- ##
==========================================
+ Coverage 96.44% 96.73% +0.28%
==========================================
Files 108 108
Lines 7509 7498 -11
==========================================
+ Hits 7242 7253 +11
+ Misses 267 245 -22
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has fixed the issue for me.
Although this makes the plot appear again, which fixes the issue, I have got some instances of a reported value of chi2 of around 10^27, which just doesn't seem right. |
For |
Fixed in cfd1c08. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, seems to work for the resonator spectroscopy (http://login.qrccluster.com:9000/Li931HqmSzO2xCvEhTwtmw==) but only together with #742.
Indeed, probably I could have made one PR point to the other. Thanks for checking! |
Hotfix for rabi length following #744
Fixing issue spotted by @igres26 in qiboteam/qibolab_platforms_qrc#119 (comment).
The problem was caused by a side effect introduced in #707.
Checklist:
master
main
main