-
Notifications
You must be signed in to change notification settings - Fork 607
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
Default implementation for preprocess_transforms and setup_execution_config #6632
base: master
Are you sure you want to change the base?
Conversation
…ss and the TOML module
This reverts commit 93b7883.
1ad15b3
to
e2d543e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6632 +/- ##
=======================================
Coverage 99.65% 99.65%
=======================================
Files 455 455
Lines 42936 42998 +62
=======================================
+ Hits 42787 42849 +62
Misses 149 149 ☔ View full report in Codecov by Sentry. |
It just occurred to me, should I add |
The issue is that |
…o default-preprocess
…o default-preprocess
…o default-preprocess
Context:
As a follow up to #6433, implement a default preprocessing transform for devices that implements a capabilities data class.
Description of the Change:
Adds a default implementation of
preprocess_transforms
andsetup_execution_config
Benefits:
Makes it easier for device developers
Related GitHub Issues:
[sc-71719]
[sc-78760]