Ouranosinc / xscen / 23649273988

90%

Build Type

push

github

Committed by web-flow

Commit Message

Que climatological_op utilise unstack_dates  + Fixs pour PROJ 9.8 (#701)

<!-- Please ensure the PR fulfills the following requirements! -->
<!-- If this is your first PR, make sure to add your details to the
AUTHORS.rst! -->
### Pull Request Checklist:
- [ ] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #xyz
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features).
- [x] (If applicable) Tests have been added.
- [x] This PR does not seem to break the templates.
- [x] CHANGELOG.rst has been updated (with summary of main changes).
- [x] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added.

### What kind of change does this PR introduce?

Mon but premier était d'avoir la mécanique du `year_start_month`
(`winter_starts_year`) de `unstack_dates` mais dans `climatological_op`.
i.e. que les pour les moyennes saisonnières, une période comme 1980-12 -
1981-02 soit associée à 1981 pour qu'elle soit le premier item de la
moyenne hivernale 1981-2020. En regardant le code, j'ai vu que
`climatological_op` ne "unstackait" pas avec `unstack_dates`, faque je
me suis dit : pourquoi pas.

Comme on a besoin de revenir à des dates "stackées", ça a été un peu
plus compliqué que prévu. Pour ajouter `stack_dates`, qui fait l'inverse
de `unstack_dates`, la méthode la plus simple que j'ai trouvée c'était
de préserver le temps original de la source dans le `unstack_dates`, ici
je l'appelle `original_time`. Dans `climatological_op`, si
`horizons_as_dim = True`, on renomme simplement `original_time` à
`time`.
 
Cette PR a comme effet secondaire de faire que `climatological_op`
supporte maintenant les données quotidiennes. Le truc pour gérer les
années bissextiles c'est de mettre `NaT` dans le `original_time` pour
les combinaisons années/doy impossibles. Pour `climatological_op` ça
veut dire que ça fonctionne, mais ça reste un peu chambranlant de
l'utiliser avec des calendriers s... (continued)

Coverage Stats

41 of 42 new or added lines in 6 files covered. (97.62%)

1 existing line in 1 file now uncovered.

4375 of 4871 relevant lines covered (89.82%)

4.49 hits per line