You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I suggest that this behaviour is changed (this is arguably a breaking change, arguably a bugfix) to use UTC consistently everywhere. To that end, the DTZ rules in ruff could be helpful to enable.
In any case, I think it would be worth documenting this behaviour in more detail.
The text was updated successfully, but these errors were encountered:
This tripped me up for longer than I care to admit today. My workaround was to set the environment variable TZ=utc so that Python pins detected the local timezone as UTC.
Regarding this comment:
pins-python/pins/versions.py
Lines 77 to 78 in d215a8c
As far as I can tell, R pins will write the
created
datetime as a UTC time:https://github.com/rstudio/pins-r/blob/de47c3141d97020e9f4a6a2e93d5823bc649105b/R/meta.R#L50
And the functions it uses to parse datetimes also specify UTC (although the two
parse_
functions indeed strip it out).https://github.com/rstudio/pins-r/blob/de47c3141d97020e9f4a6a2e93d5823bc649105b/R/meta.R#L55-L68
I haven't looked into this too deeply though.
As far as I can tell, Python pins will just use the local datetime, e.g.
pins-python/pins/versions.py
Line 104 in d215a8c
I suggest that this behaviour is changed (this is arguably a breaking change, arguably a bugfix) to use UTC consistently everywhere. To that end, the
DTZ
rules in ruff could be helpful to enable.In any case, I think it would be worth documenting this behaviour in more detail.
The text was updated successfully, but these errors were encountered: