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

Make documentation about clock drift detection more vague to not leak implementation details #69

Open
kevincianfarini opened this issue Nov 30, 2024 · 0 comments

Comments

@kevincianfarini
Copy link
Owner

In the future we might consider that resuming coroutines every minute to check for clock drift is too frequent. For example, we might consider delaying the coroutine for half of the remaining duration several times. This would allow us to only resume delayed coroutines a handful of times to check for clock drift versus checking about 1,440 times per day per pulse.

Right now the documentation specifically mentions that coroutines will be resumed every minute to check for drift. We should be more vague about this and allow this to be an implementation detail. Perhaps the following language would suffice:

Coroutines delayed for more than one minute will occasionally be resumed to check for clock drift.

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

1 participant