-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
RFC remove some of our examples #27151
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
Comments
As a part of #26927 we're also removing some examples or improving them when needed. |
ping @ArturoAmorQ who is interested in this topic and might already have looked at aggregating some info. +1 on removing examples then. We already have a way to redirect HTML pages as well. |
@glemaitre as I mentioned also in #27108 (comment) , I'm not sure how much we should care about the redirections. I'd be totally okay with giving 404, especially since the only way users could get that 404 would be if they changed the URL manually. We don't provide any way for users to see the same page in a different version, and caring about a good redirect is making it really hard to remove examples. |
Hi @lorentzenchr, I would like to work on clustering examples, having reviewed, my observations and proposed changes are as follows:
Could you advise if these would suffice or there is room to remove more. (#29937) |
These sound good suggestions @saldanhad |
Hi @adrinjalali, appreciate your feedback, will submit a PR for these suggested changes soon. |
Working on the example Plot randomly generated classification dataset. |
Edit: Merged. |
@virchan It was merged |
@glemaitre @ArturoAmorQ I would like to take Comparison of Calibration of Classifiers and Probability Calibration curves. Can I work on PR? |
Those two examples in particular are pretty nice, not sure if we need to do anything about those ones? |
@raviteja-ganta maybe you can merge Comparison of LDA and PCA 2D projection of Iris dataset and Incremental PCA, which are quite similar. |
@ArturoAmorQ Looks like both are different. First one tries to compare LDA with PCA while second one(Incremental PCA) tries to compare between PCA and incremental PCA. |
Uh oh!
There was an error while loading. Please reload this page.
TLDR: I think we have too many examples, in particular in
On top, several examples don't render a plot in the gallery, e.g. Release Highlights 1.0.0.
I would like to discuss how we can improve this situation. My main worry is that, currently, a user does not find what she/he is looking for but gets overwhelmed by the sheer amount of examples.
Edit: See task list in #27151 (comment).
The text was updated successfully, but these errors were encountered: