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
DM-33497: Improve support for quantum-backed butler #764
Conversation
Codecov ReportBase: 84.91% // Head: 84.89% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #764 +/- ##
==========================================
- Coverage 84.91% 84.89% -0.03%
==========================================
Files 267 267
Lines 35021 35052 +31
Branches 5983 5990 +7
==========================================
+ Hits 29739 29758 +19
- Misses 4000 4009 +9
- Partials 1282 1285 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
a6d3865
to
a6b9e10
Compare
The source butler in transfer_from is now a `LimitedButler` to support transfer from `QuantumBackedBUtler`. If input data IDs are not expanded we need access to registry, so it has to resort to dynamic type checking using `getattr`.
Move that option definition to `cli.opt` to enable its reuse by a new CLI command in `pipe_base` package.
a6b9e10
to
a2af9f9
Compare
A number of small changes needed for implementation of new pipetask command that use quantum-backed-butler.
Checklist
doc/changes