sockless-coding / garo_wallbox

Garo wallbox - Home Assistant Component
MIT License
22 stars 40 forks source link

Energy Reading in HS add up wrongly #19

Closed MichaelRV82 closed 2 months ago

MichaelRV82 commented 3 years ago

Hi,

this is a great integration. I am experiencing the following problem: when adding the total energy (kwh) as ”track individual energy cunsomptions” in the Energy settings in HS the reading seem to wrongly add up adding the total consumption every day instead of showing the delta.

See screenshot!

Am I doing something wrong?

/Michael 5F8A6CEA-5053-487F-B329-0E3CA661C31B

sockless-coding commented 3 years ago

Hi @MichaelRV82, Yeah, that looks off... what version of HA are you on? Does it show the same numbers if you add it as a Grid consumption?

MichaelRV82 commented 3 years ago

My core is on Version core-2021.10.6

Also looks off when I selected it for Grid Consumption. See attached.

Any ideas?

![Uploading 9BA9386C-95C4-4B74-BDD0-AB57F0936FB3.png…]()

MichaelRV82 commented 3 years ago

8E350415-CC19-4EA6-A19E-94DCBCDBFF3B

sockless-coding commented 3 years ago

The energy usage looks like it dropped to 0... that could explain the strange reading you are getting because that would cause a delta of what ever your current total is. What software version are you running on the wallbox? There is an energy readout bug in 6.9-146 and earlier.

MichaelRV82 commented 3 years ago

Good call! I just updated. 1.3.1 now.

see below. Readings still look strange?! Any ideas?

maybe I need to reset the values in HA? How is that done if possible?

Thanks for looking into it! 4D22AE98-3B99-4D03-851D-7A3F51654CC8 . 3FEDDB5E-2C96-4EC4-82CB-EB9A4ABC18C5

6A5E40D4-BFE9-4EDB-8090-ED29AF302531

StarkJohan commented 3 years ago

I'm also experiencing the same issue. Haven't had time to look into it yet. I just checked the db entries for the daily total and from time to time there is an entry of the all time total in the data set messing up the daily total just like in your example.

On Mon, Oct 25, 2021, 22:36 Michael @.***> wrote:

Good call! I just updated. 1.3.1 now.

see below. Readings still look strange?! Any ideas?

maybe I need to reset the values in HA? How is that done if possible?

Thanks for looking into it! [image: 4D22AE98-3B99-4D03-851D-7A3F51654CC8] https://user-images.githubusercontent.com/92013548/138766712-a4197e2b-c849-40db-b889-3daab9904891.png . [image: 3FEDDB5E-2C96-4EC4-82CB-EB9A4ABC18C5] https://user-images.githubusercontent.com/92013548/138766723-a34103a7-d070-46b8-910e-7668717f232f.png

[image: 6A5E40D4-BFE9-4EDB-8090-ED29AF302531] https://user-images.githubusercontent.com/92013548/138766703-69cb6781-25c1-4d6d-bf40-afda58ba1c37.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sockless-coding/garo_wallbox/issues/19#issuecomment-951306376, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDB46AS5ZDRM7USEXPW7CDUIW5UVANCNFSM5GVU7ABA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

sockless-coding commented 3 years ago

@MichaelRV82 the old values under energy will not get fixed. You have to reset them, not sure how you can do that. But keep an eye on the new readings while charging in case it spikes again.

MichaelRV82 commented 3 years ago

It is still behaving strangly:

46962281-D4B2-4D8F-AC44-0A9B19591C45

MichaelRV82 commented 3 years ago

maybe this is the key:

https://developers.home-assistant.io/docs/core/entity/sensor/#long-term-statistics

MichaelRV82 commented 3 years ago

after reseting the old values. It keeps doing it. It looks like the consumption is peaking to strange values from time to time:

32C025DE-135F-4A82-9207-2491016686F0

I would really appriciate any ideas to get it fixed!

sockless-coding commented 3 years ago

The sensor is set as total increasing, so that is not the issue. There must be something strange coming from the wallbox... If you look at the Wh sensor does it show the same strange spikes as the kWh one?

StarkJohan commented 3 years ago

I'm pretty sure the problem is false data from the wallbox. A workaround is filtering for outliers in HA. Not the solution but better than not being able to use the data at all.

On Wed, Oct 27, 2021, 20:16 Jimmy Everling @.***> wrote:

The sensor is set as total increasing, so that is not the issue. There must be something strange coming from the wallbox... If you look at the Wh sensor does it show the same strange spikes as the kWh one?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sockless-coding/garo_wallbox/issues/19#issuecomment-953190261, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDB46DJWQYM2EZB2IRQTNDUJBFZZANCNFSM5GVU7ABA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

sockless-coding commented 3 years ago

Yeah, something is off, there was an issue earlier this year... the charge power were returning a massive value every now and again. Guess there is something similar happening from your wallboxes. But I can add a check in case the total delta is larger then 3MWh then ignore that value.

StarkJohan commented 3 years ago

In my case the off values is equal to the all time total of the charger (built in meter).

On Wed, Oct 27, 2021, 20:35 Jimmy Everling @.***> wrote:

Yeah, something is off, there was an issue earlier this year... the charge power were returning a massive value every now and again. Guess there is something similar happening from your wallboxes. But I can add a check in case the total delta is larger then 3MWh then ignore that value.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sockless-coding/garo_wallbox/issues/19#issuecomment-953205316, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDB46E64ZCGAXHLKCPWDLTUJBIBNANCNFSM5GVU7ABA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

sockless-coding commented 3 years ago

@StarkJohan the Total Energy (kWh) should show the same value as the Metervalue under Energy in the wallbox web UI.

sockless-coding commented 3 years ago

I've added a check for invalid total energy values, and released v1.0.11. Any changes with a delta greater than 500 kWh are discarded.

MichaelRV82 commented 3 years ago

Thanks!!! just installed it, will keep you posted!