-
Notifications
You must be signed in to change notification settings - Fork 22
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
docs: Allow to have dropdown menu to show the API of reporters #1302
Conversation
I let the PR in this stage for the moment. If anyone wants to take over, please do not hesitate. |
@sylvaincom What do you think? |
I like what I see, thanks |
@auguste-probabl did you want to take this over? Or should I? |
I can take it DW |
@sylvaincom @MarieS-WiMLDS I've been expanding on this PR, and I found something which I need your opinion on: the API index page (https://skoredoc.k.probabl.dev/1302/reference/index.html) no longer lists all the public class/functions using the As far as I can tell, we can't have both a summary table with everything, and drop-downs in the sidebar. Which do you prefer? @glemaitre Feel free to correct me if there's a way to do it |
Yes exactly |
We might be able to increase the depth of the toc tree but indeed it will not be as nice as what we had before. I think that I would be fine with a slightly deeper toc tree since that if you click on it, you will end up in a subsection. Perfectly, in the future, I would be happy that we have a manual generation of all public methods as in scikit-learn: https://scikit-learn.org/stable/api/index.html However, I'm also OK if someone has stronger opinion on this one. |
+1 |
closes #1095
It is a wip PR to show that we can get a sort of dropdown menu with several level to show the API documentation of reporters, without the feeling that there is so much information at the top level.
The trick is to have a directory structure:
that creates the tree on the navigation sidebar. The not ideal thing is the content of
report.rst
. We need to think what content we want in this intermediate page. However, I think it is a good start to explore.A couple of screen:
(I obviously broke the API auto-generated documentation but it should be possible to fix it).