-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add 'step' and 'barstacked' to histogram_histtypes demo #15357
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
Add 'step' and 'barstacked' to histogram_histtypes demo #15357
Conversation
Add the step histogram type to the histtype demo
I had a look at the CircleCI logs and there are a lot of errors which relate to files which are not changed in this pull request. |
ax0.set_title('stepfilled') | ||
fig, axs = plt.subplots(nrows=2, ncols=2) | ||
|
||
axs[0, 0].hist(x, 20, density=True, histtype='stepfilled', facecolor='g', alpha=0.75) |
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.
This line is too long, hence the test error
./examples/statistics/histogram_histtypes.py:30:80: E501 line too long (85 > 79 characters)
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.
Oops, I have fixed the line. The CircleCI build was failing once I did this due to a missing import fixed in 4677fd9
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.
Thanks for the contribution. This mixes two documentation styles: The original two examples are listed in bullet points at the top; the two new examples instead have a comment in the code. Please unify these.
…m_histtypes demo
PR Summary
Add example plots for the
step
andbarstacked
histogram types to thehistogram_histtypes
demo. This partially addresses the discussion in #7121 by adding examples of all four available histtypes.PR Checklist