repro: make recursive by default (kill -P) #5944
jorgeorpinel
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
BugReportdvc status
by default checks all dvc.yaml/lock files recursively in the project, and in some docs we state this is a way to check whetherdvc repro
is needed. However,repro
by default doesn't do this; It needs the-P/--all-pipelines
flag to have this behavior. I see 2 issues:repro
to be recursive just like status.repro
just for a specific subfolder, as paths are not accepted astargets
byrepro
. You'd have to specify the name of the last stage of the pipeline in a subdir, but willrepro
find it without-P
? And with-P
it will probably repro everything anyway (not limited to the pipeline you want) — I need to double check this part.--all-pipelines
seems ambiguous. Do we mean all the pipelines in a single dvc.yaml file, since it can have multiple ones? Or all the pipelines in all the dvc.yaml files (current behavior)?Solution: I think repro should behave like status, and keep -P for the other meaning (all pipelines in a dvc.yaml file). Or maybe deprecate -P since the default behavior I think is already to repro all pipelines when no targets are given.
P/s supporting outputs as targets (and granularity maybe?) of repro would also be nice. Related: #4191
Please provide information about your setup
Output of
dvc version
:Beta Was this translation helpful? Give feedback.
All reactions