-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add ridge_map to third party packages documentation #14119
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
If you run optipng on it (no specific optimization level), you can save 40%. Also, cropping the outer white border might help a little (though perhaps not as much after optimization.) |
Thats a pretty cool package. 👍 for putting it on the list. |
Thanks for advice -- you were eerily correct on how much space it saved!
|
you'll want to squash the first commit too, so that the old version doesn't stay in the git history. |
We can squash merge it, and I might push an even smaller image too. Can you make a release on PyPI too? |
I shrunk it down some more by simply re-creating it closer to the size it would end up being on the page (364 ~= 400 px, anyway.) |
That'll look pretty fuzzy on high-dpi displays. |
We don't really have double-size for any other example plots. |
True - it actually looks kind of bad that we don't do this. All our figures look pretty fuzzy in our docs. |
Let's handle that in a separate PR then. |
Thanks for the prompt and friendly review! Added some CI and cut a release on PyPI last night: https://pypi.org/project/ridge-map/ |
PR Summary
This adds the
ridge_map
library (https://github.com/ColCarroll/ridge_map) to https://matplotlib.org/thirdpartypackages/index.html . I tried to make the example image as small as possible while still looking nice, but it still weighs 286kb. I can try some more tricks if that is too big!Also no problem if this is a bad fit -- the other mapping libraries in here are very impressive and scientifically useful, while this is closer to an art project (which is important, but different!)