-
Notifications
You must be signed in to change notification settings - Fork 18
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
date_range: update freq string #504
Conversation
Well no - we already test with pinned (even older) versions of xarray and pandas. So can merge on green. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #504 +/- ##
===========================================
- Coverage 87.90% 50.32% -37.59%
===========================================
Files 40 50 +10
Lines 1745 3527 +1782
===========================================
+ Hits 1534 1775 +241
- Misses 211 1752 +1541
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Oh thank you, almost started this myself. Warnings look good! |
Thanks |
* date_range: update freq string * add freq conditionals
* add mesmer_m example script * changelog * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * remove unused imports * adjustments * replace script with notebook * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * restructure and add some headings * remove inputs * add some comments * typos * date_range: update freq string (#504) * date_range: update freq string * add freq conditionals * reference: only year as link (#505) * nits * more info on time coordinate * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update comment & clear output --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Mathias Hauser <[email protected]> Co-authored-by: Mathias Hauser <[email protected]>
* date_range: update freq string * add freq conditionals
* add mesmer_m example script * changelog * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * remove unused imports * adjustments * replace script with notebook * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * restructure and add some headings * remove inputs * add some comments * typos * date_range: update freq string (MESMER-group#504) * date_range: update freq string * add freq conditionals * reference: only year as link (MESMER-group#505) * nits * more info on time coordinate * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update comment & clear output --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Mathias Hauser <[email protected]> Co-authored-by: Mathias Hauser <[email protected]>
Avoid some warnings in the tests. I am pretty sure
"YE"
is also fine for older versions of pandas/ xarray but should probably run with #503 merged, so marking as draft for now.