Skip to content

DOC refer to plot in the scatter plot doc - #7096

Merged
efiring merged 2 commits into
matplotlib:masterfrom
NelleV:7083_doc_scatter_vs_plot
Sep 20, 2016
Merged

efiring merged 2 commits into
matplotlib:masterfrom
NelleV:7083_doc_scatter_vs_plot

Conversation

@NelleV

@NelleV NelleV commented Sep 12, 2016

Copy link
Copy Markdown
Member

Encourage users to use plot instead of scatter when all markers are identical in size and colour.

Closes #7083

Here is a preview of the rendered docstring in Sphinx. Note that the "Notes" section is not rendered properly (see #7095 )

scatter_doc

Encourage users to use plot instead of scatter when all markers are identical in size and colour.

Closes #7083
@NelleV NelleV changed the title DOC refer to plot in the scatter plot doc [MRG] DOC refer to plot in the scatter plot doc Sep 14, 2016

See Also
--------
plot : to plot scatter plots when markers are identical in size and

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.

what about for scatterplots where markers are identical in size and color (nitpicky as all out, but I find the plot: to plot construction clunky)

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.

This needs backticks to cross-link, doesn't it?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No it doesn't, thought it is currently impossible to check on matplotlib's documentation.

@tacaswell tacaswell added this to the 2.0.1 (next bug fix release) milestone Sep 14, 2016
@efiring
efiring merged commit 684861b into matplotlib:master Sep 20, 2016
@efiring

efiring commented Sep 20, 2016 •

Copy link
Copy Markdown
Member

Backported to v2.x as 5ffd20d . Thank you.

@QuLogic

QuLogic commented Sep 20, 2016

Copy link
Copy Markdown
Member

@efiring I think maybe you forgot to push that?

efiring added a commit that referenced this pull request Sep 21, 2016
[MRG] DOC refer to plot in the scatter plot doc
@efiring

efiring commented Sep 21, 2016

Copy link
Copy Markdown
Member

Pushed now. Thanks for catching that!

@QuLogic QuLogic changed the title [MRG] DOC refer to plot in the scatter plot doc DOC refer to plot in the scatter plot doc Dec 7, 2016
@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
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.

6 participants