Skip to content

DOC refer to plot in the scatter plot doc #7096

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

Merged
merged 2 commits into from
Sep 20, 2016
Merged

DOC refer to plot in the scatter plot doc #7096

merged 2 commits into from
Sep 20, 2016

Conversation

NelleV
Copy link
Member

@NelleV NelleV commented Sep 12, 2016

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
@@ -3889,17 +3889,25 @@ def scatter(self, x, y, s=None, c=None, marker=None, cmap=None, norm=None,
----------------
kwargs : `~matplotlib.collections.Collection` properties

See Also
--------
plot : to plot scatter plots when markers are identical in size and
Copy link
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
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
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
Copy link
Member

efiring commented Sep 20, 2016

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

@QuLogic
Copy link
Member

QuLogic commented Sep 20, 2016

@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
Copy link
Member

efiring commented Sep 21, 2016

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