-
Notifications
You must be signed in to change notification settings - Fork 6
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
Python reference docs are almost empty #1439
Comments
In the recent doc restructuring these pages were no longer included. Related to #1439, but does not fix that issue.
I added some docstring under Model() class, and quartodoc can show those docstring now, the bug is no longer there. |
When I was working on this issue, I hope that quardoc can collect all the self-defined functions in Model class and show it in API. I tried My current conclusion is that we need to list all the functions manually, and later if new functions are added, keep in mind to also add them to API page. I think we can generate a list of functions in python, and copy paste that list to |
It might be good to link to an open issue on quartodoc about this, and link to the configuration page, where you got the |
FWIW, after checking the documentation, and this issue, we could do this:
The options will make sure to render everything (classes/attributes/etc), while the ribasim section would render everything we export. |
Thanks for the input @evetion What you put in the sections should be modules in ribasim otherwise you get error However, if I remove lines after |
Part of #1439 . I added the public functions (the one that doesn't start with a "_") to the documentation. Give some short description on some of them.
Continue work of this issue is here #1571 This issue thus can be closed |
https://deltares.github.io/Ribasim/reference/python/
Could be due to changes in https://github.com/machow/quartodoc
Possibly related to #219
Edit: I think it is working fine, but we just don't have many docstrings anymore. And currently methods that have docstrings don't seems to get included, so we should look into that.
Perhaps best done after #1298.The text was updated successfully, but these errors were encountered: