-
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
guide: add "Best Practices" #72
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Should this be merged with #230 and featured in the #899 epic? We're trying to avoid so many sections now. Also, the Questions part of What is DVC? (currently in https://dvc.org/doc/user-guide/what-is-dvc/collaboration-issues#questions) probably overlaps with this. |
@jorgeorpinel Yeah, that indeed seems suitable. |
@jorgeorpinel how would it looks like? like a subsection in How To? |
Just a single document under How To. I updated the description of this issue and in fact I think #230 is already included here, in the "manually editing dvc.yaml + dvc commit or dvc repro" checkbox. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
More:
|
Looking at open check boxes, most or all of these topics are addressed I think (the relevant ones at least).
This would be a how-to, but is it still something we want to have official docs for? Not really matching DVC's approach
@efiop is this some sort of bootstrapping method? Is it really something people do? What problem does it solve?
We do stress the use of
We mention non-deterministic behavior in general in https://dvc.org/doc/command-reference/stage/add#avoiding-unexpected-behavior and avoiding ad hoc file naming for versioning is a core use case.. |
So we could close this ticket. That said we still don't have a "Best Practices" section or guide(s). Do we want to? Maybe Use Cases or the trails of the Get Started already cover this need (informing users of the main/recommended patterns for DVC project setup/usage). WDYT @shcheklein @dberenbaum ? Thanks |
I'm fine to close this. Not all items are covered though and yes, we could have done a good page that contains tips/faq for pipelines, for data, general project structure (e.g. never store user credentials in the DVC project config) ... |
UPDATE: Possibly as a How To guide (see #899)Looks like we need a special section describing ways how to organize your projects:
our defaultDvcfile
trickdvc commit
ordvc repro
(see also how to: "update your pipeline by editing DVC-files" #230 (comment))it's safe to edit DVC files, no need to touch or update
md5
, DVC will take care of itUPDATE: See
dvc.yaml
: encourage manual writing/editing stages (and pipelines) #2578specify meaningful stage names with-f
moving it to different data setscreating a pipeline in a 'debug' directory and then modifying respective DVC files toset different data sets as an inputSee also the latest relevant #72 (comment) and below.
The text was updated successfully, but these errors were encountered: