search

scikit-hep / awkward

Manipulate JSON-like data with NumPy-like idioms.
https://awkward-array.org
BSD 3-Clause "New" or "Revised" License
850 stars 89 forks source link

fix: correct handling of keepdims and mask_identity for weighted mean #3291

Closed nj-vs-vh closed 2 weeks ago

nj-vs-vh commented 2 weeks ago

Fixes #3285

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.17%. Comparing base (b749e49) to head (b01afc9). Report is 182 commits behind head on main.

Additional details and impacted files | [Files with missing lines](https://app.codecov.io/gh/scikit-hep/awkward/pull/3291?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | Coverage Δ | | |---|---|---| | [src/awkward/operations/ak\_mean.py](https://app.codecov.io/gh/scikit-hep/awkward/pull/3291?src=pr&el=tree&filepath=src%2Fawkward%2Foperations%2Fak_mean.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep#diff-c3JjL2F3a3dhcmQvb3BlcmF0aW9ucy9ha19tZWFuLnB5) | `88.46% <ø> (+1.22%)` | :arrow_up: | ... and [158 files with indirect coverage changes](https://app.codecov.io/gh/scikit-hep/awkward/pull/3291/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep)
jpivarski commented 2 weeks ago

@all-contributors please add @nj-vs-vh for code

allcontributors[bot] commented 2 weeks ago

@jpivarski

I've put up a pull request to add @nj-vs-vh! :tada: