-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add a 'median' calculation #9
Comments
The calculation of the median requires storing in memory all variants of the values. And this can very negatively affect the work of HA. I will try to come up with something, but I doubt that it will work out. |
There was a problem with the calculations. Using the median formula, I want a complete list of all values. But this sensor, in fact, operates not with individual values, but with areas (with a geometric figure of complex shape). I have not yet been able to figure out how to calculate the median here… |
Hi Limych,
Thanks for the reply. The work around I found was to use esphome. It has a median filter on the sensor component. I’ve now decided though to use better sensors that removes the need for filtering.
Cheers,
… On 11 Sep 2020, at 4:12 am, Limych ***@***.***> wrote:
There was a problem with the calculations.
Using the median formula, I want a complete list of all values. But this sensor, in fact, operates not with individual values, but with areas (with a geometric figure of complex shape). I have not yet been able to figure out how to calculate the median here…
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#9 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJYSKJE5TFMVATYNWFGJLETSFEJKHANCNFSM4LA4DGEQ>.
|
Nice. Then I close this issue as no longer relevant. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Hi Limych, is it possible for you to add a 'median' calculation to this integration?
The text was updated successfully, but these errors were encountered: