NotificationsYou must be signed in to change notification settings
Fork7.9k
Star21.3k

Add mplcairo to 3rdparty docs.#10645

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

phobson merged 1 commit intomatplotlib:masterfromanntzer:mplcairo

Mar 2, 2018

Merged

Add mplcairo to 3rdparty docs.#10645

phobson merged 1 commit intomatplotlib:masterfromanntzer:mplcairo

Mar 2, 2018

Conversation

Copy link

Contributor

anntzer commentedMar 2, 2018•
edited
Loading

Also reorder other mentioned 3rdparty packages to alphabetical order.

PR Summary

PR Checklist

Has Pytest style unit tests
Code is PEP 8 compliant
New features are documented, with examples if plot related
Documentation is sphinx and numpydoc compliant
Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

anntzer added the Documentation label

Mar 2, 2018

anntzer added this to thev2.2-doc milestone

Mar 2, 2018

Add mplcairo to 3rdparty docs.

300fcb9

Also reorder other mentioned 3rdparty packages to alphabetical order.

anntzer force-pushed themplcairo branch from4006008 to300fcb9Compare

March 2, 2018 01:34

Copy link

Member

phobson commentedMar 2, 2018

Why'd you moveplotnine down? if anything, theggplot entry should be removed since it appears abandoned any not as complete asplotnine

Copy link

ContributorAuthor

As mentioned in the message: to keep things in alphabetical order. I don't have any experience as to whether ggplot should be removed from the list, but I'd rather not start a "recommendation" system for 3rd-party packages beyond listing or not listing them.