-
Notifications
You must be signed in to change notification settings - Fork 37
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
Update docs #86
Comments
dvclive
: Update "Introduction"
That is definitely related with iterative/dvc.org#2443, in the sense that we need to figure out how much overlap we want between the README here and the docs in https://dvc.org/doc/dvclive
This we don't want to do. @daavoo based on that other issue, would it be possible to get a summary of the changes you think would be needed on each side? We'll figure it out later if not cc @pared 🙂 |
This comment has been minimized.
This comment has been minimized.
I see, I would need to review iterative/dvc.org#2443 in order to propose that summary of changes.
That issue seems to be focused on the changes needed in the "README" but not the ones needed in the "docs". Not sure if we should keep a separate issue for each "target" or not |
Yup, now that https://github.com/iterative/dvclive/pull/99is done (thanks @daavoo) I guess we can keep this one to update the DVCLive docs index (currently in https://dvc.org/doc/dvclive). We'll need to come up with the plan first (based on iterative/dvc.org#2443) 🙂 |
So here is a granular proposal of changes for
Users can't reach this repository from the docs.
The docs index only mention the log files in
I think the ones added in #71 better reflect the library comparing to those currently present in the docs index
Establish a single point for reference as this is currently duplicated in several places (some of those with outdated info iterative/dvc.org#2498): https://github.com/iterative/dvclive#quick-start
Ref: #87 .
Document usage of existing "self-hosted" ( iterative/dvc.org#2509 (comment) What do you think @jorgeorpinel ¿? |
LGTM! Do we want to keep all of that in one place (this issue?) I think Add command/API reference might a bigger issue than other ones. Also, in this point, do you think about removing other references once this single point is established? I am asking because those three seem to have a little bit different puprose. |
I think having this issue is helpful, but also having separate issues and PRs for the individual checkboxes linking back to here would be great. |
Should we create those issues in |
I wouldn't remove the other references but rather update tthe content with a link to the docs API Reference. Probably, for each of those other references, some of the parts overlapping with the docs API reference could be removed and update the remaining parts focusing on the specific purpose |
I think we decided recently to keep that here especially if you will be the main contributor (cc @jorgeorpinel in case I'm remembering incorrectly). |
Yep! Better to track in the corresponding product repo even if it's currently documented in dvc.org/doc. |
The existing "front page" of
dvclive
seems to be missing some important points that are mentioned in the README of the repository . At the very least, we could opt for syncing both sources of information by updating the existingdvc.org
introduction with the information present in theREADME
.UPDATE: Jump to #86 (comment)
Update docs index #116
Add Command/API Reference #108
API Reference enhancements #117
Simplify Usage Guide #119
Add docs for Integrations #123
The text was updated successfully, but these errors were encountered: