Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

add time zones section to 'time series' #234

Merged
merged 11 commits into from
Feb 11, 2023

Conversation

MarcoGorelli
Copy link
Contributor

@MarcoGorelli MarcoGorelli commented Jan 23, 2023

Time zones page:

image

Addition to the bottom of the parsing dates page:

Screenshot_20230204_111847

@@ -0,0 +1,23 @@
import polars as pl

Copy link
Member

Choose a reason for hiding this comment

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

Can we split all these snippets to single files? This will ensure the line offsets are not getting messed up when we change/remove/add snippets.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yup, good call!

I've done that, these are all now in their own time_zones folder

Waiting for a release to include pola-rs/polars#6649, then I'll update to use that instead of tz_localize

@MarcoGorelli MarcoGorelli marked this pull request as ready for review February 10, 2023 10:03
@MarcoGorelli MarcoGorelli changed the title WIP: add time zones section to 'time series' add time zones section to 'time series' Feb 10, 2023
@MarcoGorelli MarcoGorelli marked this pull request as draft February 11, 2023 09:27
@MarcoGorelli MarcoGorelli marked this pull request as ready for review February 11, 2023 09:50
@MarcoGorelli
Copy link
Contributor Author

Updated as per version 0.16.3

@ritchie46 ritchie46 merged commit d5c5347 into pola-rs:master Feb 11, 2023
@ritchie46
Copy link
Member

Thanks a lot!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants