Skip to content
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

More descriptive ylabel for Series.hist() #10485

Closed
clarkfitzg opened this issue Jul 1, 2015 · 5 comments
Closed

More descriptive ylabel for Series.hist() #10485

clarkfitzg opened this issue Jul 1, 2015 · 5 comments
Milestone

Comments

@clarkfitzg
Copy link
Contributor

s = pd.Series(np.random.randn(100))
s.plot(kind='hist')
s.hist()

Running s.plot(kind='hist') labels the y axis with the word 'Degree'. A more informative label might be 'count', 'frequency', or 'density'.

Also, it seems strange that s.plot(kind='hist') and s.hist() produce plots with different labels / gridlines.

Looks like this came from PR #7809. cc @sinhrks, @jreback

@shoyer
Copy link
Member

shoyer commented Jul 2, 2015

The name "Frequency" sounds more appropriate to me -- unless normed=True, in which case "Density" would be the right word.

@clarkfitzg
Copy link
Contributor Author

+1 for 'Frequency'. That would make sense to me if normed=True or not. 'Density' makes me think of a probability density function.

@sinhrks
Copy link
Member

sinhrks commented Jul 2, 2015

Related to #9093. The reason I've added ylabel is for consistency with kind='kde'. I'm willing to change it if there is better word choice (of course PR is welcome!).

@clarkfitzg
Copy link
Contributor Author

Looks like kind='kde' labels it with 'Density'.

@sinhrks
Copy link
Member

sinhrks commented Jul 15, 2015

Closed by #10493. Thanks @clarkfitzg !

@sinhrks sinhrks closed this as completed Jul 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants