-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
Timestamp('now', tz=xxx) result wrong #9000
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
Comments
this should return exactly what
looks correct to me (and IIRC 0.12 was wrong) |
I think the issue is with Timestamp('now') which I didn't even know was a possibility.
|
from is where this is parsed. So that's a UTC time, which is then has a tz tacked on. I think this needs to indicate is UTC when passed back
|
I certainly think that we should change the behavior of 'now' to be consistent with .now. It doesn't make sense to be different. Agreed? |
yep |
In earlier version (0.14 and earlier), Timestamp('now', tz=xxx) will return time now in xxx time zone.
In 0.15.1, it returns the time in UTC but tagged with xxx time zone.
Example:
In version 0.12.0:
In version 0.15.1:
The text was updated successfully, but these errors were encountered: