-
-
Notifications
You must be signed in to change notification settings - Fork 26.2k
ENH Improve plot_semi_supervised_newsgroups.py
example
#21673
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
Conversation
Is this now 9s with LabelSpreading? If yes, you can put it back on, since it's escaped at in the CI at the end of the example. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @patel-zeel
Thanks a lot for approving my first PR on sklearn @adrinjalali :) |
plot_semi_supervised_newsgroups
exampleplot_semi_supervised_newsgroups.py
example
This should be an easy review, @jjerphan maybe? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR @patel-zeel
LGTM!
plot_semi_supervised_newsgroups.py
exampleplot_semi_supervised_newsgroups.py
example
Thank you for approving @thomasjpfan. |
Reference Issues/PRs
#21598
What does this implement/fix? Explain your changes.
before: ~104 seconds
after: ~9 seconds