-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add Yellowbrick to third party packages #14051
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
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.
Could you please run the png trough a compressor like https://tinypng.com/. I don't want to unnecessarily bloat the repo and serve larger images than necessary to the user.
Sure - I've resized and compressed the image. |
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.
Great, thanks!
Just waiting for CI pass.
If someone else merges, please squash so that the original large image is squashed out.
@meeseeksdev backport to v3.1.x |
…051-on-v3.1.x Backport PR #14051 on branch v3.1.x (Add Yellowbrick to third party packages)
@jklymak cough please be careful of this request in the future. |
Ooops, sorry! Is there any way to revert and remove the offending commit? |
No, the commits are public now. They could be removed but it's not nice to any forks and not really worth the effort in this case. |
I promise to read full sentences in the future |
PR Summary
Adds a blurb and link about Yellowbrick to the documentation along with our README gallery image that shows some of the visualizers that Yellowbrick produces for classification, regression, and clustering.
PR Checklist