8000 Added descriptions for line bars and markers examples by pdubcali · Pull Request #10796 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Added descriptions for line bars and markers examples #10796

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 3 commits into from
Mar 21, 2018
Merged

Added descriptions for line bars and markers examples #10796

merged 3 commits into from
Mar 21, 2018

Conversation

pdubcali
Copy link
Contributor
@pdubcali pdubcali commented Mar 15, 2018

PR Summary

From issue #8921
Added some descriptions for the following:
examples/lines_bars_and_markers/scatter_custom_symbol.py
examples/lines_bars_and_markers/scatter_hist.py
examples/lines_bars_and_markers/scatter_masked.py
examples/lines_bars_and_markers/scatter_star_poly.py
examples/lines_bars_and_markers/scatter_symbol.py

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

"""
import matplotlib.pyplot as plt
from matplotlib import pyplot as plt
Copy link
Member

Choose a reason for hiding this comment

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

Why did you change this? The previous way is the way its done almost everywhere in the codebase.

@@ -3,8 +3,11 @@
Scatter Symbol
==============

Scatter plot with symbols.
In this example clovers are used.
Copy link
Member

Choose a reason for hiding this comment

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

Combine to "Scatter plot with clover symbols"

@@ -3,6 +3,8 @@
Scatter Masked
==============

Add mask line in plot.
Copy link
Member

Choose a reason for hiding this comment

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

Too enigmatic: "mask some data points, and add a line demarking masked regions"

@@ -3,6 +3,9 @@
Scatter Hist
============

Creates histogram from scatter plot.
And adds them to the sides of the plot.
Copy link
Member

Choose a reason for hiding this comment

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

remove first period and capitalization.

@@ -3,6 +3,9 @@
Scatter Custom Symbol
=====================

Custom symbol in scatter plot.
This example is creating an ellipse
Copy link
Member

Choose a reason for hiding this comment

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

Creating a custom ellipse symbol in scatter plot.

@@ -3,7 +3,8 @@
Scatter Masked
==============

Add mask line in plot.
Mask some data points and add a line demarking
masked regions.
Copy link
Member

Choose a reason for hiding this comment

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

This is short enough to fit on a single line.

@phobson phobson merged commit f69408f into matplotlib:master Mar 21, 2018
@QuLogic QuLogic added this to the v3.0 milestone Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0