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

Review and improve logic for reading cycle delays #88

Open
ZavenArra opened this issue Jul 26, 2022 · 2 comments
Open

Review and improve logic for reading cycle delays #88

ZavenArra opened this issue Jul 26, 2022 · 2 comments
Assignees

Comments

@ZavenArra
Copy link
Contributor

ZavenArra commented Jul 26, 2022

We want to separate the concepts 'delay' and 'interval' in our code and documentation

A delay is a time period that takes place between the end of a process and the start of another process
An interval is a time period that takes place between the start of a process and the next iteration of the same process

As we process measurement cycles, the following shall be intervals

  • The time between each wake to begin measuring
  • The time between each burst
  • The time between each newly available reading on a sensor

And the following shall be delays

  • The global start up delay for the measurement cycle
  • The start up delay for each sensor

Image

@ZavenArra ZavenArra self-assigned this Jul 26, 2022
@jakehosen
Copy link
Contributor

Not causing a problem but still needs to be corrected to work properly.

@ZavenArra
Copy link
Contributor Author

I think we could consider calling 'start up delay' as 'start up interval' as well, which could express that all the configured start up delays start from the relevant setup completion time.

Global start up delay starts from completion of all sensor setup.
Individual sensor start up delay starts from the completion of setup for that particular sensor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Priority
Development

No branches or pull requests

2 participants