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.

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

Use weighted aggregated_by in our time-related preprocessors #1841

Open
schlunma opened this issue Nov 30, 2022 · 0 comments
Open

Use weighted aggregated_by in our time-related preprocessors #1841

schlunma opened this issue Nov 30, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@schlunma
Copy link
Contributor

Since iris==3.3.0, Cube.aggregated_by supports weights. This function is used very often in our time-related preprocessors (e.g., annual_statistics, monthly_statistics, daily_statistics, etc.), but currently does not use weights (=length of the time interval).

We could/should add weights to these preprocessors to make them more accurate (e.g., annual_statistics should consider different number of days for different months).

I'll try to tackle this for v2.8, but not sure if I can make it in time.

Relies on solving ESMValGroup/ESMValTool#2945

@schlunma schlunma added the enhancement New feature or request label Nov 30, 2022
@schlunma schlunma added this to the v2.8.0 milestone Nov 30, 2022
@schlunma schlunma self-assigned this Nov 30, 2022
@schlunma schlunma modified the milestones: v2.8.0, v2.9.0 Jan 13, 2023
@schlunma schlunma modified the milestones: v2.9.0, v2.10.0 May 16, 2023
@schlunma schlunma removed this from the v2.10.0 milestone Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant