-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add a "density" kwarg to hist2d #11070
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
Comments
Hi @dstansby, I would like to do this. This PR is for both replacing and depreciating, right? |
I think you need to deprecate the |
I’d argue for consistency with numpy. If they don’t used notmed I guess we shouldn’t. |
Agree with @jklymak (maybe I should convince numpy should change for API consistency 😉 ) |
numpy 1.15 introduced a 'density' kwarg to hist2d (numpy/numpy#11529). Even if we don't require numpy 1.15, we can always just introduce the 'density' kwarg and pass it as 'normed' to numpy. |
Hi All, |
Closed by #13128. |
Recently
ax.hist
got a density keyword argument to conform to numpy kwargs - it would be good ifax.hist2d
could have it'snormed
argument replaced with adensity
argument for API consistency.The text was updated successfully, but these errors were encountered: