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.

Sign up for GitHub

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

Jump to bottom

normalisation of cf.histogram #794

Open
JonathanGregory opened this issue Jul 11, 2024 · 2 comments · May be fixed by #795
Open

normalisation of cf.histogram #794

JonathanGregory opened this issue Jul 11, 2024 · 2 comments · May be fixed by #795
Labels
enhancement New feature or request
Milestone
NEXT VERSION

Comments

@JonathanGregory
Copy link
Collaborator

It would be convenient if cf.histogram had options to normalise the result by (a) dividing by the total of counts, giving probability in each bin, or (b) also dividing by the width of each bin, giving the probability density function. Please excuse me if I have overlooked an existing method for this.
@JonathanGregory JonathanGregory added the enhancement New feature or request label Jul 11, 2024
@davidhassell
Copy link
Collaborator

Dear Jonathan,

As we discussed offline, this is good idea, and is easy to implement. I shall add create a PR shortly.

@JonathanGregory
Copy link
Collaborator Author

Thank you!

@davidhassell davidhassell linked a pull request Jul 16, 2024 that will close this issue
Open
@davidhassell davidhassell added this to the NEXT VERSION milestone Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
NEXT VERSION
Development

Successfully merging a pull request may close this issue.

New keyword parameter to cf.histogram: density davidhassell/cf-python
2 participants
@JonathanGregory @davidhassell