-
Notifications
You must be signed in to change notification settings - Fork 26
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
Implement reset feature for mid-circuit measurements #507
Conversation
cc7dfb0
to
cd5f8d7
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #507 +/- ##
=======================================
Coverage 99.55% 99.55%
=======================================
Files 43 43
Lines 7866 7870 +4
Branches 537 537
=======================================
+ Hits 7831 7835 +4
Misses 18 18
Partials 17 17 ☔ View full report in Codecov by Sentry. |
a97b236
to
db74f1e
Compare
db74f1e
to
f9c8b45
Compare
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 👍
f9c8b45
to
b849cec
Compare
The coverage does seem bugged, I think we can merge as is 👍 |
Context: We want to support in Catalyst the existent reset feature of the measure function of PennyLane.
Description of the Change: The logic is implemented at the measure function right after the instantiation of the MidCircuitMeasure object. It applies a PauliX gate to the qubit, but only if the measurement result was 1.
Benefits: No need to involve MLIR and runtime on the process.
TODO:
[sc-56696]