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.

Sign up for GitHub

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

Jump to bottom

metrics/params/exp: should the --show-* flags have a show prefix at all? #6488

Closed
skshetry opened this issue Aug 25, 2021 · 0 comments · Fixed by #6711
Closed

metrics/params/exp: should the --show-* flags have a show prefix at all? #6488

skshetry opened this issue Aug 25, 2021 · 0 comments · Fixed by #6711
Labels
A: cli Related to the CLI ui user interface / interaction

Comments

@skshetry
Copy link
Member

skshetry commented Aug 25, 2021 •
edited
Loading

In dvc metrics/params/exp, we have show-json/show-md/show-csv flags.

Should we just have --json/--markdown/--md/--csv flag instead? This is much easier to type, show is redundant here.

eg: dvc exp show --show-json

We can easily have an alias in the argparse for the flags for backward compatibility till 3.0.
@skshetry skshetry added A: cli Related to the CLI ui user interface / interaction labels Aug 25, 2021
@skshetry skshetry changed the title metrics/params/exp: should the --show-* flags not have a show prefix at all? metrics/params/exp: should the --show-* flags have a show prefix at all? Aug 25, 2021
@skshetry skshetry mentioned this issue Sep 30, 2021
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: cli Related to the CLI ui user interface / interaction
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

cli: add short alias for --show-{csv,json,md} skshetry/dvc
1 participant
@skshetry