Skip to content

Conversation

@rcomer
Copy link
Member

@rcomer rcomer commented Jan 4, 2023

🚀 Pull Request

Description

... since PERCENTILE has lazy aggregation.

Closes #4039. Specifically #4039 (comment)

let's wait till 2023-01 if anyone really wants MEDIAN to be lazy.
IF not, add a doc note to say "it's not lazy, but percentile is"


Consult Iris pull request check list

@rcomer rcomer force-pushed the doc-median-lazy-percentile branch from 24a2923 to eec5430 Compare January 4, 2023 16:21
@trexfeathers
Copy link
Contributor

image

😍

@trexfeathers trexfeathers merged commit f190415 into SciTools:main Jan 4, 2023
@rcomer
Copy link
Member Author

rcomer commented Jan 4, 2023

@rcomer rcomer deleted the doc-median-lazy-percentile branch January 4, 2023 18:09
tkknight added a commit to tkknight/iris that referenced this pull request Jan 5, 2023
* upstream/main:
  Unpin theme (SciTools#5129)
  link percentile as lazy option (SciTools#5128)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Which statistics in iris.analysis are lazy?

2 participants