Skip to content
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

run: make --name a regular argument and allow multiple args for -d/-o #4172

Closed
jorgeorpinel opened this issue Jul 6, 2020 · 3 comments
Closed
Labels
discussion requires active participation to reach a conclusion enhancement Enhances DVC

Comments

@jorgeorpinel
Copy link
Contributor

The current minimal use of dvc run requires both --name/-n (option) and a command (argument). I believe -n is in fact one of the few or maybe the only required option in all of our commands. How about make it into a proper argument?

$ dvc run [--options] <name> <command>

E.g.

$ dvc run train -d data -d train.py -o model python train.py

We'd may need to keep -n around (as an option to the name arg) for backward compatibility, though.


Another idea is to allow -d/--deps and -o/--outs to accept several option args, to avoid repeating the flags. This already applies to -p/--params, via commas.

E.g.

$ dvc run featurize \
       -d data/clean,feats.py \
       -p f.range,f.limit \
       -o data/feats,data/map.csv \
          /feats.py
@jorgeorpinel jorgeorpinel added enhancement Enhances DVC discussion requires active participation to reach a conclusion labels Jul 6, 2020
@jorgeorpinel jorgeorpinel changed the title run: make --name a regular argument? run: make --name a regular argument and allow multiple args for -d/-o Jul 6, 2020
@efiop
Copy link
Contributor

efiop commented Jul 7, 2020

@jorgeorpinel Great idea! We didn't do that yet because dvc run is too overloaded already and I hope that we will have something like dvc stage add(or something else) in the future, where it would have a more friendly interface. Let's keep this issue open for now.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jul 7, 2020

Agree. dvc stage [--options] <name> <cmd> to create/update (maybe even --remove instead of dvc remove) stages, and separately dvc run to run/repro stages and pipelines seems ideal for the 1.0 era IMO (1.2 I guess).

@efiop
Copy link
Contributor

efiop commented Dec 8, 2023

Closing as stale

@efiop efiop closed this as not planned Won't fix, can't repro, duplicate, stale Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion requires active participation to reach a conclusion enhancement Enhances DVC
Projects
None yet
Development

No branches or pull requests

2 participants