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

F1: RTCAdjustment example add incremental calibration #753

Conversation

ag88
Copy link
Contributor

@ag88 ag88 commented Feb 15, 2020

Incremental calibration allows the user to call calibratertc() repeatedly.
This is done by running synctime() once and the user can then call calibratertc(() more than once.

calibratertc() computes the drift adjustments by adding up the cumulative adjustments each time the user call calibratertc(). By doing this, it accounts for the smaller differential errors detected from the elapsed time since the initial synctime() call, and hopefully provide more accurate adjustments over a longer time period.

so initially call synctime() then afterwards you can call calibratertc() more than once with the new time to sync, for each time it measures the total elapsed time difference from the initial time sync to get the differential error. And add this incremental error to the drift_duration which is basically the 1 in N seconds error with each calibratertc() measurements

Incremental calibration allows the user to call calibratertc() repeatedly.
This is done by running synctime() once and the user can then call calibratertc(() more than once.

calibratertc() computes the drift adjustments by adding up the cumulative adjustments each time the user call calibratertc(). By doing this, it accounts for the smaller differential errors detected from the elapsed time since the initial synctime() call, and hopefully provide more accurate adjustments over a longer time period.
@stevstrong
Copy link
Collaborator

As it does not change any core files I see low risk so I will commit this.

@stevstrong stevstrong merged commit 971e43b into rogerclarkmelbourne:master Dec 25, 2020
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

2 participants