-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Add weighted quantile * Add weighted quantile to documentation * Apply suggestions from code review Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> * Improve _weighted_quantile_type7_1d ufunc with suggestions * Expand scope of invalid q value test * Fix weighted quantile with zero weights * Replace np.ones by xr.ones_like in weighted quantile test * Process weighted quantile data with all nans * Fix operator precedence bug * Used effective sample size. Generalize to different quantile types supporting weighted quantiles (4-9, but only 7 is exposed and tested). Fixed unit tests * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply suggestions from code review Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> Co-authored-by: Illviljan <14371165+Illviljan@users.noreply.github.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * added missing Typing hints * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update what's new and pep8 fixes * add docstring paragraph discussing weight interpretation * recognize numpy names for quantile interpolation methods * tweak to avoid warning with all nans data. simplify test * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * remove integers from quantile interpolation available methods * remove merge artifacts * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * [skip-ci] fix bad merge in whats-new * Add references * renamed htype argument to method in private functions * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update xarray/core/weighted.py Co-authored-by: Abel Aoun <aoun.abel@gmail.com> * Add skipped test to verify equal weights quantile with methods * Apply suggestions from code review Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> * Update xarray/core/weighted.py Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> * modifications suggested by review: comments, remove align, clarify test logic * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply suggestions from code review Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * use broadcast * move whatsnew entry * Apply suggestions from code review * switch skipna determination * use align and broadcast Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com> Co-authored-by: Illviljan <14371165+Illviljan@users.noreply.github.com> Co-authored-by: David Huard <huard.david@ouranos.ca> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com> Co-authored-by: Abel Aoun <aoun.abel@gmail.com> Co-authored-by: Mathias Hauser <mathias.hauser@env.ethz.ch>
- Loading branch information
1 parent
8f42bfd
commit 8a2fbb8
Showing
5 changed files
with
453 additions
and
20 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.