Skip to content

460 etl timezone#462

Open
kjlippold wants to merge 2 commits into
release/v1.10.4from
460-etl-timezone
Open

460 etl timezone#462
kjlippold wants to merge 2 commits into
release/v1.10.4from
460-etl-timezone

Conversation

@kjlippold

Copy link
Copy Markdown
Contributor
  • Updated parse_series_to_utc to preserve timestamp timezones when provided, and handle mixed tz-aware/tz-naive timeseries.
  • Added missing argument to ETL CSVTransformer.

@kjlippold kjlippold requested a review from daniel-slaugh July 6, 2026 17:11
@kjlippold kjlippold self-assigned this Jul 6, 2026

@daniel-slaugh daniel-slaugh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran this locally and the happy path works as expected.

I found one nitpick bug: If the timezone information is hour only like 2024-01-01T12:00:00-07, timestamp.py will crash the batch since the timestamp is misclassified as naive, tries to localize, and throws a TypeError. I recommend adding support for this since we'll probably have a user at some point that uses this timestamp format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants