search

NCAS-CMS / cf-python

A CF-compliant Earth Science data analysis library
http://ncas-cms.github.io/cf-python
MIT License
119 stars 19 forks source link

New keyword parameter to `cf.histogram`: density #795

Open davidhassell opened 1 month ago

davidhassell commented 1 month ago

Fixes #794

sadielbartholomew commented 1 month ago

Wanting a review, @davidhassell?

davidhassell commented 1 month ago

Thanks, but I think Jonathan's got this one covered :)

davidhassell commented 1 month ago

Good point on the units - I'll fix that.

davidhassell commented 1 month ago

@JonathanGregory, is this better: https://github.com/NCAS-CMS/cf-python/pull/795/commits/8e5487ffcba6af534d1b38485a2da1ab515d9db1 ?

sadielbartholomew commented 1 month ago

Thanks, but I think Jonathan's got this one covered :)

Oops sorry David, I missed that part somehow and thought you'd forgot to assign anyone!