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

Preprocessor extract_season does not return complete seasons #2591

Open
k-a-webb opened this issue Nov 28, 2024 · 4 comments · May be fixed by #2597
Open

Preprocessor extract_season does not return complete seasons #2591

k-a-webb opened this issue Nov 28, 2024 · 4 comments · May be fixed by #2597
Labels
enhancement New feature or request

Comments

@k-a-webb
Copy link

When the dataset time vector starts or ends mid-season, extract_season will return incomplete seasons for the first and/or last years.
For example, if a data set spans 2000-01-01 to 2010-12-31, the season DJF will have just JF in 2000 and just D in 2010.

In the related pull request, I propose to add an additional argument to extract_season, full (default False), which will drop incomplete seasons.

@k-a-webb k-a-webb added the enhancement New feature or request label Nov 28, 2024
@valeriupredoi
Copy link
Contributor

technically correct though 🤣 I suspect there's some string slicing going on in there, it'd be great if you could have a closer look, and many thanks 🍺

@valeriupredoi
Copy link
Contributor

valeriupredoi commented Nov 29, 2024

@schlunma is there even such a thing as an "incomplete season" in the modelling nomenclature ie as Kristi mentions, is JF a valid season? (I thought not, but am a mere engineer)

@schlunma
Copy link
Contributor

schlunma commented Dec 3, 2024

Well, if it say's "DJF", all of the seasons should be there, not just "JF". So in my opinion it's reasonable to add this here.

@valeriupredoi
Copy link
Contributor

cheers, Manu! What I thought too, unless you live in Wales, where all seasons are called MR (for mild and rainy) 🏴󠁧󠁢󠁷󠁬󠁳󠁿

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

Successfully merging a pull request may close this issue.

3 participants