-
Notifications
You must be signed in to change notification settings - Fork 39
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
Update LightningQubit
to adhere to MCM qnode arguments
#736
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #736 +/- ##
==========================================
- Coverage 94.57% 92.71% -1.87%
==========================================
Files 122 15 -107
Lines 17781 1729 -16052
==========================================
- Hits 16816 1603 -15213
+ Misses 965 126 -839 ☔ 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.
Nothing to flag here. I think a couple tests needed fixing, but this looks ready to merge. I'll approve once the CI goes green.
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.
LGTM, 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.
Thanks @mudit2812 🙌
**Context:** As name says. `QNode` now accepts `postselect_shots` and `mcm_method` arguments to specify MCM behaviour. This PR updates LQ to adhere to those arguments. **Description of the Change:** * Update LQ preprocessing to adhere to user requested MCM behaviour * Add tests for LQ and LK * Update LQ and LK simulation to adhere to specified `postselect_mode`. **Benefits:** **Possible Drawbacks:** **Related GitHub Issues:** --------- Co-authored-by: ringo-but-quantum <github-ringo-but-quantum@xanadu.ai>
Context:
As name says.
QNode
now acceptspostselect_shots
andmcm_method
arguments to specify MCM behaviour. This PR updates LQ to adhere to those arguments.Description of the Change:
postselect_mode
.Benefits:
Possible Drawbacks:
Related GitHub Issues: