-
Notifications
You must be signed in to change notification settings - Fork 394
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
cmd: example on modifying any outputs into a plots #1946
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far so good! A few more comments added ☝️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple details left before we can merge this, but it looks ready! Thanks @imhardikj
@@ -30,6 +30,10 @@ Property names are passed as [options](#options) to this command (prefixed with | |||
`--`). These are based on the full | |||
[Vega specification](https://vega.github.io/vega/docs/specification/). | |||
|
|||
Note that a secondary use of this command is to convert output or simple | |||
`dvc metrics` file into a plots file (see |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dvc metrics
file
I like this, we should probably apply in other places where manual links to "plots/metrics file(s)" exist. (In another PR).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But don't worry about it @imhardikj, I'll take care of that.
Partially Addresses #1414