-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
LTC SMPTE: Improve NTP Client synchronise process. #292
Comments
Hi @dirkdamerau Dirk, I have added a new state : "LOCKED". The NTP client will run with POLL_SECONDS_MIN (8 seconds) until the NTP offset with the server is less than 1 milliseconds. This is it the "LOCKED" state. Now it is running with POLL_SECONDS_MAX (approx 1 hour and 8 minutes). With the new system time implemented in #287, this should be a pretty accurate NTP based clock. Groeten, Arjan |
Dear Arjan, I checked the image above as well as the one you uploaded today @ #294. I do not see if and when the locked state is reached. I can confirm that the sync to the NTP Server works but I still see missing and missorderd frames in the LTC Output. It happens randomly between 17s and 3:01m. So to say once and a while every few minutes. Most of the time it follows this pattern: 00000000 23:48:06:20 | 31310952 31312871 At first there is one frame missing and a second or two later there is a doubled missordered. So the overall accuracy is ok from my standpoint but the LTC output is not consistent. If running "internal" it is still pretty accurate with no errors like in the last test. Something keep the LTC Output unstable when NTP Client is running. It is ok for me looking at it but it could fail if I use it to trigger something based on it as time of day. |
Dear Dirk, thank you so much for the feedback. The discontinuity is always around the last frame. I have some thoughts why it could be; the LTC is continues running while the source local time get changed by the NTP Client. This happens in the beginning and should not happen anymore in the locked state. I will work first on #296; that should be a quick win. I am eager to get this discontinuity issue solved. Thanks, Arjan |
Follow up for #287 @dirkdamerau
Place holder; more details will be added.
The text was updated successfully, but these errors were encountered: