-
Notifications
You must be signed in to change notification settings - Fork 586
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 reference.qubit
for testing and reference
#6181
Conversation
mini.qubit
for testing and referencereference.qubit
for testing and reference
2264edf
to
20c024b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6181 +/- ##
=======================================
Coverage 99.58% 99.58%
=======================================
Files 443 444 +1
Lines 42273 42333 +60
=======================================
+ Hits 42096 42156 +60
Misses 177 177 ☔ View full report in Codecov by Sentry. |
…o mini-dev-new
reference.qubit
for testing and referencereference.qubit
for testing and reference
Co-authored-by: Christina Lee <christina@xanadu.ai>
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.
What would you think about adding some assertions inside execute
to make sure we are passing in suitably supported tapes? For example:
assert tape.batch
assert all(supports_operation(op) for op in tape.operations)
Not mandatory, but might be useful.
Co-authored-by: Christina Lee <christina@xanadu.ai>
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.
❤️
…o mini-dev-new
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.
Should this be added to the devices
so it turns up in the documentation here?
Co-authored-by: lillian542 <38584660+lillian542@users.noreply.github.com>
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 great, thanks @astralcai! 🚀
Created from #5445
[sc-65558]