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

Energy dashboard showing high values after update 5.0 and 5.1 #116762

Closed
Anonymouse0104 opened this issue May 4, 2024 · 14 comments · Fixed by #116849
Closed

Energy dashboard showing high values after update 5.0 and 5.1 #116762

Anonymouse0104 opened this issue May 4, 2024 · 14 comments · Fixed by #116849
Assignees

Comments

@Anonymouse0104
Copy link

The problem

Since the core update 5.0 (and yesterday 5.1) my energy dashboard values are ruined. It has been working for years but for somehow it reads the values wrong. The biggest misread is the PV side, but the grid side is wrong also if I check the sensor changes in de integration (DSMR).

The dashboard:
image
image

The input sensors:
image

The sensors in the integrations:
image
image
image
image
image

What version of Home Assistant Core has the issue?

2024.5.1

What was the last working version of Home Assistant Core?

2024.4.4

What type of installation are you running?

Home Assistant Core

Integration causing the issue

GoodWE and DSMR

Link to integration documentation on our website

https://www.home-assistant.io/integrations/goodwe

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

home-assistant bot commented May 4, 2024

Hey there @mletenay, @starkillerOG, mind taking a look at this issue as it has been labeled with an integration (goodwe) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of goodwe can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign goodwe Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


goodwe documentation
goodwe source
(message by IssueLinks)

@Anonymouse0104
Copy link
Author

Edit: seems to be aa GoodWE integration fault? The sensor Total PV Generation sets a 0 value every 5 minutes?
image

@fnurkje
Copy link

fnurkje commented May 4, 2024

Same issue started 2nd of may
Screenshot_20240504-161540

@mletenay
Copy link
Contributor

mletenay commented May 4, 2024

Which inverter model ?
And please provide some logs around these 0 values.

@fnurkje
Copy link

fnurkje commented May 4, 2024

Model:
GW5K-DT
GoodWe
Firmware: 909.909.09 / None

Just turned on debug logging, let you know results tomorrow
Screenshot_20240504-223539
Screenshot_20240504-223458

@mletenay
Copy link
Contributor

mletenay commented May 4, 2024

Hmmm, it seems I might accidentally break the DT inverter code in this commit marcelblijleven/goodwe@f41674c
Before that, when inverter failed to response properly (weird on its own), the value decoded to None and now it is 0.

@maxxie85
Copy link

maxxie85 commented May 5, 2024

I can confirm the same issue.
I also have a GW5K-DT on firmware 10.10.0b

@mletenay
Copy link
Contributor

mletenay commented May 5, 2024

Fix for the above mentioned problem is on the way, but what bothers me a bit is it looks exactly like this ticket #115837 , but that was reported on 2024.4.1 where there was previous version of the goodwe library ...

@Guy-Manuel
Copy link

Same problem here since 2 may, Goodwe GW8K-DT

@mletenay mletenay mentioned this issue May 5, 2024
20 tasks
@Anonymouse0104
Copy link
Author

Fix for the above mentioned problem is on the way, but what bothers me a bit is it looks exactly like this ticket #115837 , but that was reported on 2024.4.1 where there was previous version of the goodwe library ...

Same Inverter and software, I will wait untill the fix

@mletenay
Copy link
Contributor

mletenay commented May 5, 2024

Sorry for the troubles caused, but it seems there is a bug/problem with DT inverters that they sometimes report 0xFFFF/-1 as value of the total power/energy registers.
And recent fix in underlying library which addressed problems with power values on large inverters changed decoding behavior, which was (unintentionally) working around this (not known) DT problem.

To be honest, I have no idea how to fix your energy dashboard data. If my assumptions are correct, the 0 values should be removed and the rest recalculated ? Not sure how to do it or if it is possible at all ...

This fix is already implemented and is waiting for review.
If you cannot or do not want to wait for next HA release, the fix is already present in v0.9.9.18 of HACS version of this integration - https://github.com/mletenay/home-assistant-goodwe-inverter

@maxxie85
Copy link

maxxie85 commented May 5, 2024

There are 2 ways to fix the dashboard.
Restore the backup you hopefully made before the upgrade from 2024.4.

Another way would be via the development tools, statistics. Then find the correct statistics entry. And there you can adjust incorrect entries. But this is a manual job. It does have a tool to show outliers to find them easy, but you still need to manually calculate the correct value.

@Guy-Manuel
Copy link

Guy-Manuel commented May 6, 2024 via email

@sfstar
Copy link

sfstar commented May 7, 2024

For anyone reading this ticket.
The calculated sum value of the statistics starts massively increasing after a 0 value is reported.
In order to fix this you can install the sqlite addon select the records for your goodwe entities metadata_id and export them.
Fix them in excel and delete the records in HA and import them again + a restart.
Unfortunately I found the developer tools statistics to be lacking when it came to correcting the values of the goodwe entities to a correct value.
Maybe a good improvement would be to become more strict with the reset when total increasing is selected.
Since the reset values aren't being changed while HA treats the 0 being reported as a reset.

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

Successfully merging a pull request may close this issue.

7 participants