gramex icon indicating copy to clipboard operation
gramex copied to clipboard

Published 20 hours ago • verified publisher icon gramener

Can we remove tzlocal as a dependency?

Open sanand0 opened this issue 3 years ago • 1 comments

Currently, Gramex requires tzlocal.

This is specifically to convert timestamps for influxdb. But we don't do this for any other database. (Notable, we let MongoDB manage it via the tz_aware client parameter.)

It's entirely possible for the InfluxDB and Gramex instances to be on different timezones.

Can we remove this as a dependency, allowing the database or the developer to handle this?

sanand0 avatar Nov 06 '22 04:11 sanand0

Agreed, sent a PR.

jaidevd avatar Nov 07 '22 11:11 jaidevd