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

Update on NTP_DELTA value in ntptime.py #4661

Closed
wants to merge 1 commit into from
Closed

Update on NTP_DELTA value in ntptime.py #4661

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 31, 2019

The NTP_DELTA value has been changed to reflect the difference described in http://docs.micropython.org/en/v1.9.3/wipy/library/utime.html

Before the value was a difference of a whole century which does not seem to have any meaning and looks like it was an unintentional typo.

The NTP_DELTA value has been changed to reflect the difference described in http://docs.micropython.org/en/v1.9.3/wipy/library/utime.html

Before the value was a difference of a whole century which does not seem to have any meaning and looks like it was an unintentional typo.
@dpgeorge
Copy link
Member

Thanks, but I'm pretty sure it's correct as it is. The NTP timestamp is relative to 1 Jan 1900, and MicroPython's Epoch is relative to 1 Jan 2000, so that's why the number is as it is.

@dpgeorge dpgeorge closed this May 6, 2019
tannewt added a commit to tannewt/circuitpython that referenced this pull request Apr 30, 2021
Issue micropython#4661 - Wrap pulsein values when maxlen is exceeded.
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.

None yet

1 participant