-
Notifications
You must be signed in to change notification settings - Fork 35
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
Add null qubit #1179
Add null qubit #1179
Conversation
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.
Hi @AmintorDusko , everything I've seen looks good. Whenever you feel ready to open it for review, just change it so that it is not in draft mode. Cheers!
Hello. You may have forgotten to update the changelog!
|
Co-authored-by: erick-xanadu <[email protected]>
Co-authored-by: erick-xanadu <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1179 +/- ##
==========================================
- Coverage 97.93% 97.93% -0.01%
==========================================
Files 77 77
Lines 10921 10918 -3
Branches 971 971
==========================================
- Hits 10695 10692 -3
Misses 179 179
Partials 47 47 ☔ 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.
Thanks Amintor, I'll give conditional approval here pending resolution of the last comments.
Co-authored-by: David Ittah <[email protected]>
Co-authored-by: David Ittah <[email protected]>
Co-authored-by: David Ittah <[email protected]>
Context: Implement a Catalyst runtime plugin that mocks out all functions in the QuantumDevice interface and implement a PennyLane device plugin (implemented in qjit_device.py) providing access to the Null runtime device.
Description of the Change:
Benefits:
Such a device will be useful for a few reasons:
Possible Drawbacks: It is a surprise.
Related GitHub Issues:
[sc-72829] [sc-72830]