-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
DOC: move example #20624
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: move example #20624
Conversation
Move plot_permutation_test_for_classification from feature_selection to model_selection, where it feels more natural
Maybe we should add a stub example with a sphinx reference to the new location to avoid breaking incoming URLs? |
Maybe we should add a stub example with a sphinx reference to the new location to avoid breaking incoming URLs?
Sounds good. I'll also try to add a redirect, so that web searches update their links.
|
OK, this is good to go, now including a redirection. |
Just an additional thought: is there a way in |
Cc @lucyleeow |
I pushed something that should improve a bit the thumbnail in the gallery |
Things are now slightly better in the thumbnail gallery. |
Do you mean showing the default thumbnail image instead of a figure from the example? I think this would work: https://sphinx-gallery.github.io/stable/configuration.html#providing-an-image-for-the-thumbnail-image |
Do you mean showing the default image instead of a figure from the example?
Rather, not showing the example at all in the gallery
|
How is auto_examples/index.rst generated? |
How is auto_examples/index.rst generated?
By sphinx-gallery, I believe.
|
We don't tweak so many things apart of the sections ordering: https://github.com/scikit-learn/scikit-learn/blob/main/doc/conf.py#L334 |
If there is not easy way, I am OK with the current solution of @GaelVaroquaux |
😅 Yes it is. I can't see a way to prevent it being added to the index but still be converted to rst, not even an ugly hack. Would need a new config item to be added. |
OK so let's go with the current proposal. Thanks @GaelVaroquaux @lucyleeow |
Opened #20625 to improve redirections. |
Move plot_permutation_test_for_classification from feature_selection to model_selection, where it feels more natural, as discussed in #20622 with @thomasjpfan