Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use local timestamps for day summaries #72

Open
Nate-Wessel opened this issue Aug 2, 2018 · 4 comments
Open

Use local timestamps for day summaries #72

Nate-Wessel opened this issue Aug 2, 2018 · 4 comments
Labels

Comments

@Nate-Wessel
Copy link
Contributor

This will require a spatial timezone lookup when points are created. Several libraries are available. Current timezone is UTC across the board.

@Nate-Wessel Nate-Wessel added the improvement thing to improve label Aug 2, 2018
@Nate-Wessel Nate-Wessel self-assigned this Aug 2, 2018
@Nate-Wessel
Copy link
Contributor Author

A spatial lookup was extremely slow with the timezonefinder package. Implemented a config parameter for local time for the moment.

@Nate-Wessel
Copy link
Contributor Author

It might be sufficiently efficient to look up one arbitrary point in each episode and use that timezone in all points in that episode.

It would be good to time the timezone lookup function to see if the benefit is worth the likely delay.

@Nate-Wessel
Copy link
Contributor Author

If we just insist that all coordinates provide a local timezone name/offset then we can just use that in the output. There are lots of timezone boundary files that should make this pretty easy to do before hand for data that doesn't already have that set.

@Nate-Wessel
Copy link
Contributor Author

@Nate-Wessel Nate-Wessel removed their assignment Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant