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

Increases the accuracy of the synchronization by a factor of 50. #132

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zeflo
Copy link

@zeflo zeflo commented Mar 4, 2021

Until now the fraction of the second was not taken into account. Therefore the resulting time offset is up to 1000 ms. Using the provided Information increases the accuracy of the synchronization to 20-30 milliseconds. And it requires just two lines of code.

plot
In this plot both methods are compared. Plotted is the time difference in respect to a PPS signal provided by a GPS receiver. For comparison the time was synchronized over 1000 times and the resulting time difference filled into the respective histogram.

Until now the fraction of the second was not taken into account. Therefore the resulting time offset is up to 1000 ms. Using the provided Information increases the accuracy of the synchronization to 20-30 milliseconds. And it requires just two lines of code.

https://imgur.com/a/aVjvHON
In this plot both methods are compared. Plotted is the time difference in respect to a PPS signal provided by a GPS receiver.  For comparison the time was synchronized over 1000 times and the resulting time difference filled into the respective histogram.
@zeflo zeflo changed the title Update NTPClient.cpp Increases the accuracy of the synchronization by a factor of 50. Mar 21, 2021
@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
All committers have signed the CLA.

@per1234 per1234 linked an issue Jun 23, 2024 that may be closed by this pull request
@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: code Related to content of the project itself labels Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

An 0.5s System Error
3 participants