search

oceanprotocol / pdr-backend

Instructions & code to run predictoors, traders, more.
Apache License 2.0
28 stars 22 forks source link

[Dependencies] Investigate possibility to remove pytz #1291

Closed calina-c closed 3 months ago

calina-c commented 3 months ago

Suggested by @kdetry: pytz is only used in one place throughout the app. We can lighten the dependency load by removing it.

calina-c commented 3 months ago

@kdetry since you had the most recent changes to timezone awareness, can you look into this? If the natural language result is timezone aware, we might not need to convert it to UTC when doing UnixTimeMS.from_dt() but I do think there will be differences. Either way, this is a sensitive part of the ETL process so let's not rush anything just to get pytz out.