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.

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

start: Data Access mentions tracking models without clarifying what kind #2096

Closed
onavsaber opened this issue Jan 15, 2021 · 3 comments · Fixed by #2214
Closed

start: Data Access mentions tracking models without clarifying what kind #2096

onavsaber opened this issue Jan 15, 2021 · 3 comments · Fixed by #2214
Assignees
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: start Content of /doc/start good first issue Good for newcomers p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. 🐛 type: bug Something isn't working.

Comments

@onavsaber
Copy link

I was reading the documentation on Data Access. It starts by mentioning that

Okay, now that we've learned how to track data and models with DVC and how to version them with Git, next question is how can we use these artifacts outside of the project?

However, the previous tutorial Get Started: Data Versioning only shows how to track data and tracking models are not demonstrated.

@shcheklein shcheklein added 🐛 type: bug Something isn't working. good first issue Good for newcomers A: docs Area: user documentation (gatsby-theme-iterative) p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. labels Jan 15, 2021
@jorgeorpinel
Copy link
Contributor

The assumption is that the model is a file (probably). So the process would be the same as with the example data.xml use throughout https://dvc.org/doc/start/data-versioning.

@onavsaber
Copy link
Author

I see. Thanks for the clarification. I was thinking whether or not the models are handled differently from the data, for example, like MLflow.

Maybe a clarification like what you mentioned can be added.

@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Jan 17, 2021

Yep, we can definitely clarify!

@jorgeorpinel jorgeorpinel changed the title Data Access tutorial mentions tracking models without discussing it start: Data Access mentions tracking models without clarifying what kind Jan 17, 2021
@iesahin iesahin self-assigned this Feb 12, 2021
iesahin added a commit that referenced this issue Feb 24, 2021
…ke-2

Restyle start: Data Access and Data Versioning to mention Model in titles (#2096)
shcheklein pushed a commit that referenced this issue Mar 29, 2021
) (#2214)

* guide: disclaim x data (impro #2104)

* Added changes from PR #2188 and modified paths & titles

- Changes title of "Data Access" to "Data and Model Access"
- Changes title of "Data Versioning" to "Data and Model Versioning"
- Renames path of Data Access and Data Versioning to
  `data-and-model-access.md` and `data-and-model-versioning.md`
  respectively.
- Adds redirects
-- `/doc/start/data-access` -> `/doc/start/data-and-model-access`
-- `/doc/start/data-versioning` ->
`/doc/start/data-and-model-versioning`
- Replaces links in `/doc/start` with the new links.

* Update redirects-list.json with fixed subsection redirects.

Co-authored-by: Jorge Orpinel <[email protected]>

* Fixed incomplete looking sentence

* merged into a single paragraph

* Divided models sentence and added "large files" phrase.

* Adds new paths to sidebar

* Updated links to data-access and data-versioning cmd ref

* updated links to data-access and data-versioning in blog

* Updated links to data-access and data-versioning in UC

* Updated links to data-access and data-versioning in UG

* updated yarn.lock

* Update content/docs/start/data-and-model-versioning.md

Co-authored-by: Jorge Orpinel <[email protected]>

* Restyled by prettier

* fixes hardcoded links to data-and-model-access in the blog

* minor fixes

* guide: revert Exp Outs guide rename
per #2154 (review)

* start: emphasize models are files (assumption)

* start: roll back unnecessary changes

unnecessary for #2214

Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: Emre Sahin <iex@levinas>
Co-authored-by: Restyled.io <[email protected]>
@iesahin iesahin added the C: start Content of /doc/start label Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: start Content of /doc/start good first issue Good for newcomers p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. 🐛 type: bug Something isn't working.
Projects
None yet
4 participants