-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: Add unit tests for older timezone issues #21491
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
TST: Add unit tests for older timezone issues #21491
Conversation
@mroeschke looks awesome! so can you either add these issue numbers to the whatsnew for #21407, or if its a 'different' issue (even though it was solved via the same), add an appropriate whatnew e.g. a resampling fixed deserves its own whatsnew (in bug fixes for all of these i think) |
Codecov Report
@@ Coverage Diff @@
## master #21491 +/- ##
=======================================
Coverage 91.92% 91.92%
=======================================
Files 153 153
Lines 49587 49587
=======================================
Hits 45583 45583
Misses 4004 4004
Continue to review full report at Codecov.
|
thanks @mroeschke ! |
- Bug in :class:`Timestamp` constructor where passing an invalid timezone offset designator (``Z``) would not raise a ``ValueError``(:issue:`8910`) | ||
- Bug in :meth:`Timestamp.replace` where replacing at a DST boundary would retain an incorrect offset (:issue:`7825`) | ||
- Bug in :meth:`DatetimeIndex.reindex` when reindexing a tz-naive and tz-aware :class:`DatetimeIndex` (:issue:`8306`) | ||
- Bug in :meth:`DatetimeIndex.resample` when downsampling across a DST boundary (:issue:`8531`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should put those in 0.23.2, as we didn't fix them in there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see #21407
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These all didn't get fixed by #21407 per se. These issues may have been fixed by other prior PR's in other releases. What should our whatsnew policy be on cleaning up these older issues that have already been fixed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if it is needed to list them at all. But if we list them, I would rather add them eg to the 0.24.0 (or 0.23.0) file than to the 0.23.2 file. For those bug fix releases, the list is much shorter and it is often more valuable to actually look which bugs/regressions have been fixed compared to 0.23.0 (in the 0.24.0 or 0.23.0 files the list is huge anyways ...)
it’s fine to move some to 0.24 whatsnew |
Sounds good. I have another timezones issues PR in the works so I'll keep that in mind |
git diff upstream/master -u -- "*.py" | flake8 --diff
In the spirit of your #21407 (comment) @jreback, cleaned up some old timezone issues (but not necessarily covered by #21407) that appear to have been solved