-
-
Notifications
You must be signed in to change notification settings - Fork 18.3k
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
AmbiguousTimeError exception when resampling near DST change date #8744
Comments
cc @rockg We apparently have 4 open issues on almost the same exact thing. Can you guys looks thru and recommend dispositions for each of these (e.g. close as dupe of another, separate distinct issue). All other things equal, we generally keep the oldest issue open. |
I think #5694 might be fixed. I don't get any NaNs like I used to and there was a commit to add timezones so the offsets work properly which fixed it. Nevertheless, we can start adding some tests for that. I did take a look at both #5172 and #5694 and have some comments in both. I remember reaching a place where I didn't know where to proceed, but I can take another crack at it. I think for starters we can move all these examples to the original. Clearly they all are related to resample and probably just different incarnations of the same thing at first glance...if we find they are substantially different we can open separate issues at that point. @ischwabacher, what do you think? |
closing as dupe of #5172 for now |
ok, think I fixed up all the issues and made #5172 the master issue |
Thank you, I'll keep watching #5172 |
UTC offset changed from +8 to +7 at 2:00 local time.
The text was updated successfully, but these errors were encountered: