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

api: automatic generation #3595

Closed
daavoo opened this issue May 27, 2022 · 2 comments
Closed

api: automatic generation #3595

daavoo opened this issue May 27, 2022 · 2 comments
Labels
A: website Area: website C: ref Content of /doc/*-reference p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. type: discussion Requires active participation to reach a conclusion.

Comments

@daavoo
Copy link
Contributor

daavoo commented May 27, 2022

Extracting new issue from #2770 , as the Python API Reference (kind of) is a separate topic from CLI.

... for Python API, we could use the docstrings content and it could be considered a way of "enforcing" detailed and up-to-date docstrings on the core repo, which are very helpful when using the API inside code editors.

Originally posted by @daavoo in #2770 (comment)

@daavoo
Copy link
Contributor Author

daavoo commented May 27, 2022

We might begin to expand DVC Python API so debating this becomes more relevant.

As a potential example of a complete docstring see iterative/dvc@6180d28 .

So, @iterative/docs :

  • what are the main points for duplicating content (or worse, the current asymmetric state), when limiting the scope to Python API)?

  • How could this be automated (from code docstrings to docs website)

  • If the problem of automation is the cost of infrastructure changes, could the Python API Reference live on a separate site (i.e. hosted in github pages associated with dvc repo) ?

IMO, long docstrings are not an issue in Python world so points regarding CLI in (#2770 (comment)) don't really outweigh the cost of manual sync efforts.

Complete docstrings make a more pleasant experience for API users (i.e. IDE integrations) and match the status of the ecosystem/

@shcheklein
Copy link
Member

I think there was initial discussion and the conclusion that we didn't want to have a very comprehensive API descriptions (the way you made in that commit). If now consider including everything docs need, it's a different story.

Then we will have the first time situation where we'll have some content rendered on the website that comes from the doc strings. Totally fine, but workflow will become a bit more complicated. At the very least I'm not sure if it will be possible to create preview environment for this, will it require first updating the main repo + running some scripts to do MD in the second one.

@shcheklein shcheklein added type: discussion Requires active participation to reach a conclusion. A: docs Area: user documentation (gatsby-theme-iterative) C: dvclive Content of /doc/dvclive A: website Area: website and removed A: docs Area: user documentation (gatsby-theme-iterative) labels May 27, 2022
@jorgeorpinel jorgeorpinel changed the title Python API automatic generation of ref. api: automatic generation Jun 2, 2022
@jorgeorpinel jorgeorpinel added the p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. label Jun 2, 2022
@jorgeorpinel jorgeorpinel added C: ref Content of /doc/*-reference and removed C: dvclive Content of /doc/dvclive labels Sep 8, 2022
@dberenbaum dberenbaum closed this as not planned Won't fix, can't repro, duplicate, stale Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: website Area: website C: ref Content of /doc/*-reference p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. type: discussion Requires active participation to reach a conclusion.
Projects
None yet
Development

No branches or pull requests

4 participants