-
Couldn't load subscription status.
- Fork 706
Qnode primitive captures unprocessed ExecutionConfig
#8258
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
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.
Thanks for fixing this!
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8258 +/- ##
==========================================
- Coverage 99.40% 99.39% -0.01%
==========================================
Files 551 551
Lines 57214 57215 +1
==========================================
Hits 56871 56871
- Misses 343 344 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Nice
Co-authored-by: Andrija Paurevic <[email protected]>
Context:
Blocks PennyLaneAI/catalyst#2041
Description of the Change:
The capture
Device.setup_execution_configwas about the capabilities of the capture execution pipeline, which is no longer maintained or actively developed. While I'm not sure where will be the best place for this logic in catalyst, we can leave that up to catalyst and no determine it in program capture.Benefits:
Unblocks catalyst unification work.
Possible Drawbacks:
Related GitHub Issues:
[sc-99193]