Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Add note to hist docstring about speed#15375
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
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.
When, you‘re done, please squash your changes into a single commit to keep the history clean.
Uh oh!
There was an error while loading.Please reload this page.
f20b6d7
to2526b4d
CompareThanks! |
PR Summary
Add note to hist docstring stating that using
histtype='bar'
can be slow when using a large number of bins. I have used the wording proposed here:https://github.com/matplotlib/matplotlib/pull/12493/commits. As#12493 involves adding a new rcparam and has had no comments for almost a year I have made this a separate pull request in order to resolve the discussions in#7121.PR Checklist