-
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
cases: Versioning Data and Models (rewrite) #1747
Conversation
This comment has been minimized.
This comment has been minimized.
b7378a9
to
df7a5f6
Compare
explain why versioning large files is important/a thing per #1716 (comment)
df7a5f6
to
87264eb
Compare
d400921
to
87264eb
Compare
It's good in a sense that we are starting it from scratch 👍 Regarding the intro- it still does not give any sense why would versioning data and models is important (reference to a problem), how DVC solves it and why DVC is the way to go (selling the philosophy). |
This comment has been minimized.
This comment has been minimized.
6421771
to
8bdae1d
Compare
content/docs/use-cases/versioning-data-and-model-files/index.md
Outdated
Show resolved
Hide resolved
is important, and how it looks with DVC per #1747 (comment)
content/docs/use-cases/versioning-data-and-model-files/index.md
Outdated
Show resolved
Hide resolved
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.
Another iteration of this is ready for review. Here are 2 known (minor) issues I see:
content/docs/use-cases/versioning-data-and-model-files/index.md
Outdated
Show resolved
Hide resolved
b628516
to
1efd8fa
Compare
1efd8fa
to
6a75449
Compare
f261c69
to
2ccd32e
Compare
complicated paths like `data/20190922/labels_v7_final` or for constantly | ||
editing these in source code. | ||
|
||
- **Efficient data management**: Use a familiar and cost-effective storage |
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 final ask in private to update this a bit
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.
Cool 😎 stuff!
@jorgeorpinel would you still like me to review this for SEO-related updates, as you mentioned before? |
That would be useful @jeremydesroches but completely up to you. Also at this point, since the rewrite was merged recently, do you think we should wait more weeks to be able to compare search patterns vs the previous version? I guess giving it a read to make sure the key terms you added in https://github.com/iterative/dvc.org/pull/1806/files#diff-34b5f307478d996f5f763b6e71d8e9689543d9ef99f85d36a505c8ef47e57fee are still there (if possible) would be ideal. |
UPDATE: Pending stuff