Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC/TST : document and test negative width to bar #4196

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
Apr 6, 2015

Conversation

tacaswell
Copy link
Member

ax.bar allows you to specify the left edge of the bars. The
'align' keyword allows the user to tell mpl to interpret the left
edges as the centers. To use the left edges as the right edges the
user can simply pass a negative width.

  • Added note to the 'align' entry in the docstring
  • Added test to make sure this does not get 'fixed' with validation on
    width in bar

@tacaswell tacaswell added this to the next point release milestone Mar 6, 2015
@@ -1849,6 +1849,8 @@ def bar(self, left, height, width=0.8, bottom=None, **kwargs):
If `edge`, aligns bars by their left edges (for vertical bars) and
by their bottom edges (for horizontal bars). If `center`, interpret
Copy link
Member

Choose a reason for hiding this comment

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

Since we are fixing the docstring here, shouldn't edge and center have regular quotes and not backticks, since they are string values, not arguments?

`ax.bar` allows you to specify the left edge of the bars.  The
'align' keyword allows the user to tell mpl to interpret the left
edges as the centers.  To use the left edges as the right edges the
user can simply pass a negative width.

 - Added note to the 'align' entry in the docstring
 - Added test to make sure this does not get 'fixed' with validation on
   width in bar
@tacaswell
Copy link
Member Author

@jenshnielsen Rebased, it was whitespace issues in test_axes.py

jenshnielsen added a commit that referenced this pull request Apr 6, 2015
DOC/TST : document and test negative width to bar
@jenshnielsen jenshnielsen merged commit 6e3471d into matplotlib:master Apr 6, 2015
@tacaswell tacaswell deleted the bar_doc branch May 16, 2015 03:25
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