dvc.org icon indicating copy to clipboard operation
dvc.org copied to clipboard

guide: Add `--glob` and `--recursive` options to Running Experiments

Open iesahin opened this issue 4 years ago • 5 comments

It looks there are bugs in dvc exp run related to these options. It's better to remove these options from the ongoing guide in #2690.

Waiting For

  • [x] iterative/dvc#6456
  • [ ] iterative/dvc#6457
  • [ ] ref: Revise exp run not to contain any options automatically inherited from dvc repro

iesahin avatar Aug 18 '21 08:08 iesahin

It looks #6457 isn't going forward, so I'll just add --recursive to the guide.

iesahin avatar Sep 22 '21 14:09 iesahin

So will https://github.com/iterative/dvc.org/pull/2846 close this?

jorgeorpinel avatar Sep 25 '21 01:09 jorgeorpinel

I couldn't decide if it closes. It may be better to remove dvc repro options statement in the usage text dvc exp run command reference and list the supported dvc repro options only.

iesahin avatar Sep 27 '21 10:09 iesahin

Sorry @iesahin can you clarify your answer? I did not understand how that's related to this issue about the guide.

Also did you just add

Revise exp run not to contain any options automatically inherited from dvc repro

or was that check box always there? Why would that be and how is it related here again? I'm a bit lost... can you please re-scope this issue and/or create other ones so they're clear? Thanks

jorgeorpinel avatar Sep 28 '21 05:09 jorgeorpinel

OK I checked the PR and I think we don't need to cover --recursive in the Running Exps guide, just like we don't cover all the repro options in it (I think, right?). This is more related to data pipelines.

list the supported dvc repro options only

Let's definitely create an issue about this. What options are we listing in exp run that are actually not supported?

Thanks

jorgeorpinel avatar Sep 28 '21 06:09 jorgeorpinel