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
-
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