Skip to content

Conversation

jcjveraa
Copy link
Contributor

While the pandas dataframe seems to be in UTC (my inspector shows 'Z' for zulu timezone, =UTC), Using datetime.datetime.timestamp seems to interpret the time as the local time. This results in different outputs depending on your local timezone (clearly visible in the ' Time of Day signal plot' ).

Using the pandas equivalent function works uniformly independent from local environment.

While the pandas dataframe seems to be in UTC (my inspector shows 'Z'  for zulu timezone, =UTC), Using datetime.datetime.timestamp seems to interpret the time as the local time. This results in different outputs depending on your local timezone (clearly visible in the ' Time of Day signal plot' ).

Using the pandas equivalent function works uniformly independent from local environment.
@google-cla
Copy link

google-cla bot commented Dec 23, 2020

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no CLA has not been signed label Dec 23, 2020
@jcjveraa
Copy link
Contributor Author

@googlebot I signed it!

@google-cla google-cla bot added cla: yes CLA has been signed and removed cla: no CLA has not been signed labels Dec 23, 2020
@8bitmp3 8bitmp3 added the review in progress Someone is actively reviewing this PR label Jun 3, 2021
Copy link
Contributor

@8bitmp3 8bitmp3 left a comment

Choose a reason for hiding this comment

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

@github-actions github-actions bot added the lgtm Community-added approval label Jun 8, 2021
@8bitmp3 8bitmp3 added ready to pull Start merge process lgtm Community-added approval and removed lgtm Community-added approval review in progress Someone is actively reviewing this PR labels Jun 8, 2021
@TensorFlow-Docs-Copybara TensorFlow-Docs-Copybara merged commit ac1efa8 into tensorflow:master Jun 11, 2021
@8bitmp3
Copy link
Contributor

8bitmp3 commented Jun 11, 2021

Thanks again @jcjveraa 👍 =

@jcjveraa
Copy link
Contributor Author

jcjveraa commented Jun 11, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes CLA has been signed lgtm Community-added approval ready to pull Start merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants