-
Notifications
You must be signed in to change notification settings - Fork 402
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
DOC Split out API reference docs #1174
Comments
I checked what scikit-learn does for this since they have exactly what we want in a working state. I think I got something promising with just "autosummary". I pushed it to a little branch. Feel free to build on that if you want! https://github.com/fairlearn/fairlearn/tree/romanlutz/separate_page_per_class_and_func |
Oh np, autosummary was another way I saw to do this! If that's working then I will close this issue :) |
TBH I don't know about pros and cons of each option. |
This is exactly the issue I was referring to @alliesaizan! About a million years ago I opened an issue for this #720, I will close that one in favor of this new issue. |
Gotcha, I will apply the autosummary method to all the modules and send a PR! |
This looks like it's completed. @hildeweerts can you close if you agree? This technically goes all the way back to #455 so I want to make sure it's addressed in a way that's consistent with your expectations 🙂 |
Similar to @adrinjalali I'm not a huge fan of listing everything on the left hand side, but we can look into that in the future if it's really bothering people. Will close this issue! |
Describe the issue linked to the documentation
Currently, the API docs for each module include all the components of that module on one page, as shown below:
Suggest a potential alternative/fix
Suggest splitting the docs up so that each component in the module gets its own page, like in the image below:
This issue was brought up during the weekly community call on 1/12/23, and I found a package that can automatically split up Sphinx docs, sphinx-audomodapi.
Looping in @hildeweerts @MiroDudik @romanlutz to see if this package would work for this use case (also @hildeweerts, let me know if I misinterpreted the issue you mentioned!). I'd be happy to research this more and submit a PR if you think this would be a good solution!
The text was updated successfully, but these errors were encountered: