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
DOC improves documentation on the pyplot module and the bar method #1757
Conversation
FWIW #1760 fixes some of the whitespace issues not addressed in this PR (so no conflicts 😄 ) |
or sequences | ||
Parameters | ||
---------- | ||
left : sequence of scalars |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sequence or scalars? Or was that intentional? It's no longer clear that scalars may be given.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was intentional, but that because I forgot the case of the horizontal bar plot, where one may want to pass a scalar here.
Looks good to me. Glad to have MEP10 rolling in. |
horizontal bars, *align* = 'edge' aligns bars by their bottom | ||
edges in bottom, while *align* = 'center' interprets these | ||
values as the *y* coordinates of the bar centers. | ||
linewidth : scalar or sequence of scalar, optional, default: None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing an 's': scalar or sequence of scalars
Is it possible to restart this Travis job? The option is greyed out for me. |
@dmcdougall I've rebased. This should restart travis. |
DOC improves documentation on the pyplot module and the bar method
Good work @NelleV (again)! |
This patch contains some MEP10 documentation improvements on the bar's method, and some small PEP8 fixes on the pyplot module.
Thanks,
N