Small `dynamic_one_shot` change to account for Catalyst updates

PennyLaneAI / pennylane

PennyLane is a cross-platform Python library for quantum computing, quantum machine learning, and quantum chemistry. Train a quantum computer the same way as a neural network.

https://pennylane.ai

Apache License 2.0

2.17k stars 568 forks source link

Small `dynamic_one_shot` change to account for Catalyst updates #5888

Closed mudit2812 closed 1 week ago

mudit2812 commented 1 week ago

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:

github-actions[bot] commented 1 week ago

Hello. You may have forgotten to update the changelog! Please edit doc/releases/changelog-dev.md with:

A one-to-two sentence description of the change. You may include a small working example for new features.
A link back to this PR.
Your name (or GitHub username) in the contributors section.

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.66%. Comparing base (2a8019f) to head (b1c151d).

Additional details and impacted files

```diff @@ 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 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.