Ouranosinc / xclim / 5415813563

90%

main: 92%

push

github

Convert doys (#1406)

<!--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:
- [x] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #1283 
- [x] Tests for the changes have been added (for bug fixes / features)
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features)
- [x] CHANGES.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?

* New `convert_doy` function to convert the calendar reference of doy
data. Two options :
- "year" : doy is seen as the fraction of the year. I.e. : 360 in
`360_day` converts to 365 in `noleap`.
- "date" : doy is seen as a date (MM-DD). 360 in `360_day` converts to
364 in `noleap` (December 30th).
* Convert calendar can automatically apply `convert_doy` if its new
argument `doy=True` (or `doy='date'`).
* `convert_doy` will NOT convert the time coordinate.

### Does this PR introduce a breaking change?
No.

### Other information:

31 of 35 new or added lines in 1 file covered. (88.57%)

8231 of 9079 relevant lines covered (90.66%)

3.49 hits per line