-
Notifications
You must be signed in to change notification settings - Fork 585
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
Small dynamic_one_shot
change to account for Catalyst updates
#5888
Conversation
Hello. You may have forgotten to update the changelog!
|
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.
Looks good to me, I'm glad to see this is coming in. Thanks @mudit2812 .
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.
🎉
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5888 +/- ##
==========================================
- Coverage 99.67% 99.66% -0.01%
==========================================
Files 425 425
Lines 40753 40457 -296
==========================================
- Hits 40620 40323 -297
- Misses 133 134 +1 ☔ View full report in Codecov by Sentry. |
**Context:** Some updates are being made to Catalyst for mid-circuit measurements that are dependent on the changes in this PR. **Description of the Change:** * Change reference in `dynamic_one_shot` to catalyst's `MidCircuitMeasure.out_classical_tracers` to `MidCircuitMeasure.mcm_tracer`. * Change `MidMeasureMP.name` to be more general. This is because `MidCircuitMeasure` will soon inherit `MidMeasureMP`, and this change makes more sense than adding a `name` property to `MidCircuitMeasure`. **Benefits:** **Possible Drawbacks:** **Related GitHub Issues:**
**Context:** Some changes were added preemptively in #5888 for changes that were being added to Catalyst in PennyLaneAI/catalyst#832, but that PR is not merged, so the changes from #5888 have to be reverted. **Description of the Change:** Reverted changes from #5888 Some updates to catalyst-`split_non_commuting` caused test failures in Catalyst in cases where terminal measurements contain mid-circuit measurements, so `qml.equal` was updated to handle abstract measurement values. **Benefits:** **Possible Drawbacks:** **Related GitHub Issues:**
Context:
Some updates are being made to Catalyst for mid-circuit measurements that are dependent on the changes in this PR.
Description of the Change:
dynamic_one_shot
to catalyst'sMidCircuitMeasure.out_classical_tracers
toMidCircuitMeasure.mcm_tracer
.MidMeasureMP.name
to be more general. This is becauseMidCircuitMeasure
will soon inheritMidMeasureMP
, and this change makes more sense than adding aname
property toMidCircuitMeasure
.Benefits:
Possible Drawbacks:
Related GitHub Issues: