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

Doesn't work if system time is changed #4

Open
jduranf opened this issue Jun 10, 2019 · 4 comments
Open

Doesn't work if system time is changed #4

jduranf opened this issue Jun 10, 2019 · 4 comments

Comments

@jduranf
Copy link

jduranf commented Jun 10, 2019

If system time is changed forwards, the PPS generator stars sending We are late this time... continuously. This can be fixed by replacing ts_expire_req by ts_expire_real in this line.

@jsln
Copy link
Owner

jsln commented Jun 29, 2019

Thanks for the input, I guess if system time is changed forwards, you see the "We are late this time..." for a while before we catch up, right? I will keep this into account on the next update.

@jduranf
Copy link
Author

jduranf commented Jul 1, 2019

Yes. Take in account that my suggestion fix the issue when the time is changed forwards, but not when the time is changed backwards.
Also, another suggestion is to change the "We are late this time..." from error message to debug message. When the module is working fine, we see this message every several seconds.

@Roozter
Copy link

Roozter commented Sep 16, 2019

Will I have issues if I use this generator while my system is being synchronized with an external GPS?

@jsln
Copy link
Owner

jsln commented Sep 18, 2019

Will I have issues if I use this generator while my system is being synchronized with an external GPS?

I used it this way and I did not notice any issue related to the GPS capture.

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

No branches or pull requests

3 participants