-
Notifications
You must be signed in to change notification settings - Fork 393
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
ref: misc. improvements around "stage" concept #3235
Conversation
stage add
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.
Explanation of some changes 👇
## Typical DVC workflow | ||
|
||
- In an existing Git repository, initialize a <abbr>DVC project</abbr> with | ||
`dvc init`. | ||
- Copy data files or dataset directories for modeling into the repository, and | ||
track them with DVC using the `dvc add` command. | ||
- Process the data with your own source code, using `dvc.yaml` and/or the | ||
`dvc stage add` command to specify further <abbr>outputs</abbr> that should |
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.
Ended up rewriting this whole section 😅
Freeze [stages](/doc/command-reference/stage) until `dvc unfreeze` is used on | ||
Freeze [stages](/doc/command-reference/run) until `dvc unfreeze` is used on |
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.
These are for consistency (only place where stage
ref is linked insted of run
) for now. Wil be properly addressed in #2883.
_dvc stage add --plots file.csv ..._ command assign the default template that | ||
needs to be changed in many cases. A simple command changes the template: | ||
Something like `dvc stage add --plots file.csv ...` assigns the default |
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.
We can auto-link to https://dvc.org/doc/command-reference/stage/add#--plots directly from here now.
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.
One type, otherwise looks good!
Co-authored-by: Dave Berenbaum <[email protected]>
* ref: rewrite Typical Workflow in index page rel. #3223 (comment) * ref: link "stage" concept to `run` ref. for now rel. #3223 (comment) * ref: stage add --plots autolink from plots modify per #3223 (comment) * Update content/docs/command-reference/import-url.md Co-authored-by: Dave Berenbaum <[email protected]> Co-authored-by: Dave Berenbaum <[email protected]>
Per #3223 (review)