Ribasim icon indicating copy to clipboard operation
Ribasim copied to clipboard

Datetimes need a timezone

Open evetion opened this issue 2 years ago • 1 comments

datetimes need a timezone

That would probably need a separate discussion / design. If we start this then I guess all our datetimes always will need a timezone, or we need to set it in the config.

Originally posted by @visr in https://github.com/Deltares/Ribasim/issues/887#issuecomment-1856024734

evetion avatar Dec 14 '23 22:12 evetion

TOML support local date time and offset date time. Currently we use local. We could support offset date time for starttime and endtime, also as a way to indicate to consumers of our data on how to interpret our time zone naive times if they have to.

The GeoPackage spec confuses me a bit, it supports a DATETIME that must be in UTC, but the time columns we have right now are TIMESTAMP and of the form 2020-01-01 00:00:00, which is time zone naive.

Arrow timestamps optionally support a timezone, and are time zone naive otherwise.

visr avatar Feb 22 '24 14:02 visr