-
Notifications
You must be signed in to change notification settings - Fork 586
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
legacy device: check if object is measurement before checking stopping_condition #5071
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5071 +/- ##
==========================================
- Coverage 99.67% 99.66% -0.01%
==========================================
Files 394 394
Lines 35670 35428 -242
==========================================
- Hits 35554 35310 -244
- Misses 116 118 +2 ☔ View full report in Codecov by Sentry. |
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.
🚀
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.
Love it when its such an easy fix :)
Same as #5071 but in other places. I'm a little worried that maybe we shouldn't deprecate `MeasurementProcess.name` at all... seems like we kinda depend on it? My first thought was to just make them more correct (eg. "State", "Probs"), but I don't think any devices have those saved in their `dev.observables` so I suppose this might still be the best way forward
…g_condition (#5071) **Context:** Many plugins are failing because of the deprecation of `MeasurementProcess.name` (#5047). The old device API checks if an object meets the stopping requirement _or_ if it's a measurement (in that order). Due to this ordering, we reference `mp.name` which is now deprecated. **Description of the Change:** Swap the order of conditions in `_local_tape_expand` to avoid hitting this deprecation warning **Benefits:** Plugins will work again! **Possible Drawbacks:** N/A
Same as #5071 but in other places. I'm a little worried that maybe we shouldn't deprecate `MeasurementProcess.name` at all... seems like we kinda depend on it? My first thought was to just make them more correct (eg. "State", "Probs"), but I don't think any devices have those saved in their `dev.observables` so I suppose this might still be the best way forward
Context:
Many plugins are failing because of the deprecation of
MeasurementProcess.name
(#5047). The old device API checks if an object meets the stopping requirement or if it's a measurement (in that order). Due to this ordering, we referencemp.name
which is now deprecated.Description of the Change:
Swap the order of conditions in
_local_tape_expand
to avoid hitting this deprecation warningBenefits:
Plugins will work again!
Possible Drawbacks:
N/A