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

Feat: Average sensor value #1136

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

inckie
Copy link
Contributor

@inckie inckie commented Aug 7, 2023

Average sensor value calculation using basic continuous integration (not that one).
It's aligned to the current Min/Max values behavior, which is seems to be expected now (i.e. values are preserved since last app start/reset even if the original event is not in the timeline history anymore).

Known issues:

  • calculates averages values for parameters that do not make physical sense (like accumulated totals for traffic, etc.). We can filter these out if needed.
  • does not handle system time change in a nice way (plots will not handle it properly too, though).

image

Form was edited from code, since I can't open form editor (need to run IDE in x86 mode or something, probably).

Comments welcome.

#967

@inckie inckie marked this pull request as ready for review August 14, 2023 01:17
@inckie
Copy link
Contributor Author

inckie commented Aug 22, 2023

@PhyxionNL PR is ready for merge, any change request?

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

1 participant