X Tutup
Skip to content

Update doc/thirdpartypackages/index.rst#8538

Merged
phobson merged 1 commit intomatplotlib:masterfrom
anntzer:thirdparty
Apr 26, 2017
Merged

Update doc/thirdpartypackages/index.rst#8538
phobson merged 1 commit intomatplotlib:masterfrom
anntzer:thirdparty

Conversation

@anntzer
Copy link
Copy Markdown
Contributor

@anntzer anntzer commented Apr 26, 2017

  • Add a few more subdivisions (specialty plots, interactivity).
  • Sort packages alphabetically within subdivisions.
  • Reword a few sentences.
  • Reflow long lines.

.. image:: /_static/holoviews.png
:height: 354px

seaborn
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kinda feel like seaborn should get top priority here since it's probably the most widely used of the lot.
Then maybe group ggplot and holoview as declarative wrappers, and treat seaborn as a standalone statistical package (and I don't think it's the only one...) or maybe group it with prob-scale and pyUpSet as mathy viz.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit uncertain whether we should give seaborn a special position or not. Let's see what others think...

exist. Below we list a number of these. Note that they are each
maintained and distributed separately from Matplotlib, and will need
to be installed individually.
exist. Below we list a number of these. Note that they are each maintained
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think Below we list a number of these is sort of redundant, or at the least should be We list a number of these below. Suggested rephrase (and yes, nitpicky)

Several external packages that extend or build on Matplotlib functionality exist. They are maintained and distributed separately from matplotlib and so need to be installed individually. We list a number of these below.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Several external packages that extend or build on Matplotlib functionality are
listed below. They are maintained and distributed separately from Matplotlib
and thus need to be installed individually.

sounds good?

Copy link
Copy Markdown
Member

@story645 story645 Apr 26, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😄 Yup.

:height: 157px

Mapping Toolkits
****************
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd put these at the top since they're in some ways the most distinct, MPL does not provide but this is core visualization (whereas everything else is more flourish).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this is inconsistent with your remark on seaborn :-)
They are very distinct, but also very specialized, so I wouldn't put them at the top.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah yeah... I do think these should get higher priority though as they support the only fundamental chart types (geospatial seems to get lumped in as basic on most every survey I've seen of viz types) matplotlib really doesn't support. .

Then I'm thinking group all the math ones together: seaborn, pyUbset, mpl-prob, then group the interactive, or the declarative, and then shrugs

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good, fixed.

- Add a few more subdivisions (specialty plots, interactivity).
- Sort packages alphabetically within subdivisions.
- Reword a few sentences.
- Reflow long lines.
@phobson
Copy link
Copy Markdown
Member

phobson commented Apr 26, 2017

Would it be worth adding a blurb about plotnine in place of ggpy?

@anntzer
Copy link
Copy Markdown
Contributor Author

anntzer commented Apr 26, 2017

@phobson I asked @has2k1 to write his own PR for that :-)

@phobson phobson added this to the 2.1 (next point release) milestone Apr 26, 2017
@phobson phobson merged commit 4d9738f into matplotlib:master Apr 26, 2017
@anntzer anntzer deleted the thirdparty branch April 26, 2017 17:42
@dstansby
Copy link
Copy Markdown
Member

Worth backporting to 2.0.x?

@phobson
Copy link
Copy Markdown
Member

phobson commented Apr 26, 2017

I don't think so, but can easily be convinced otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup