home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.67k stars 30.8k forks source link

Growatt integration double counting power on hybrid inverter when power is coming from battery #80905

Closed Wingman2 closed 1 year ago

Wingman2 commented 2 years ago

The problem

I have being watching this error for some time and it's very consistent

My system is a Hybrid inverter with ~10Kwh of storage

What I see is that power coming from the battery is counted as imported from the grid during nights.

Here I show the same day from Home assistant image image

Report from Growatt image Only Importe power image

Looks like there is some misunderstanding on what's what. Power wise.

I can´t figure if there an obvious mistake adding power from source.

What version of Home Assistant Core has the issue?

2022.10.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

growatt_server

Link to integration documentation on our website

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

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[bot] commented 2 years ago

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

Code owner commands Code owners of `growatt_server` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Change the title of the issue. - `@home-assistant unassign growatt_server` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


growatt_server documentation growatt_server source (message by IssueLinks)

muppet3000 commented 2 years ago

That's really odd. I've looked through everything above and from what I can tell you have your energy dashboard configured correctly (I have the same type of system and it's configured exactly the same and it's working correctly). Can you please take a look at the statistic that is sitting behind the "Import from grid today" value in the HA overview page like this: image Maybe it's not using the correct value?

The growatt library & this integration just pull the value directly from the Growatt servers, it should be taking the value that you see on the Growatt website titled "Imported From Grid" (the yellow bar under "Load Consumption"). Remember that this integration however doesn't make queries to the Web UI, but instead makes a call to the API used by the android application.

This value is shown on the "dashboard" tab on the shinephone app (which I'll post a screen grab of later, for some reason right now the app on my phone isn't working). If you can see it on there it'll be interesting to see what value you're seeing there.

muppet3000 commented 2 years ago

Screenshot_2022-10-25-19-24-05-57_d6f8b068dc5e28159f3efb7bac2a33b7~2

This is the stat that's pulled from the app, would be interesting to compare it to what your stat in HA is showing, also please compare it against the value you're seeing in the growatt web page.

Wingman2 commented 2 years ago

Here are some new datapoints

I have checked my configuration and it looks like yours. image

When charging from the sun looks like it's counting like charging from the grid.

On the Growatt web image

On the App image

In the energy panel image Here you see clearly that the from grid item is adding power coming from the sun

check on the 9 to 10 time.

Total import on Growatt is 1,2 Kwh and in homeassistant is 2,3Kwh

homeassistant's total power usage at the end of the day is almost double. Yesterday was 23Kwh(GW page) vs 30Kwh(HA). And it makes sense since my battery is using 7,5kwh to charge. 25% is backup reserved and not in use at the moment.

muppet3000 commented 2 years ago

Ok that's very odd. I'll look into it more when I get the chance.

Currently my focus is on the other issue with Growatt: https://github.com/home-assistant/core/issues/80950

Once I've got to the bottom of that and reviewed it I'll circle back to this as it seems to be an isolated problem to just your system.

muppet3000 commented 2 years ago

Hi All,

This is a massive cross-post on all the Growatt tickets so apologies for it's generic format. I've collated all of the open issues and prioritised them here: https://community.home-assistant.io/t/growatt-integration-prioritised-list-of-features-for-implementation-fixing/483850

I will be working through them in order and updating that post as well as the affected tickets. Using that method I can be sure to correctly track everything associated with the integration.

Watch this space!

muppet3000 commented 2 years ago

@Wingman2 - In order to investigate this further I'm going to need to do some very specific debug on your system. Would you be willing to share your Growatt credentials with me via email (contact information on my Github page) so that I can spin up a debug instance and compare it against what you're seeing?

muppet3000 commented 2 years ago

Working behind the scenes with @Wingman2 I've determined this issue is related to the fact that the specific values used for this reading etouser_combined when queried from the Growatt API occasionally drop when they shouldn't (see graph below). This causes the energy reporting functionality in Home Assistant to incorrectly think the value has been 'reset' as it falls outside of the 10% threshold used to determine whether a sensor has been reset. Since this is an error that is very specific to Growatt and in particular this one statistic I will implement a fix/workaround this in the sensor itself. I am currently testing the fix on my local setup and will submit a PR when both I and @Wingman2 have concluded the problem is resolved.

sidequest-tomas commented 2 years ago

Watching this, and trying to add my battery storage to Energy dashboard but no entities show up to fit this. Also using a hybrid inverter and 10kw battery (ARK 2.5 modules). Keep the good work up! :)

This is (picture) where i can not find entities for energy going into the battery and out of the battery 👍 image

muppet3000 commented 2 years ago

Watching this, and trying to add my battery storage to Energy dashboard but no entities show up to fit this. Also using a hybrid inverter and 10kw battery (ARK 2.5 modules). Keep the good work up! :)

This is (picture) where i can not find entities for energy going into the battery and out of the battery 👍 image

This isn't the same issue, but is related to statistics on the energy dashboard, please can you tell me what type of system you have by logging into the growatt website and telling me what appears here: image

sidequest-tomas commented 2 years ago

Ok great, this is how it look image

also maby this info is needed? image

muppet3000 commented 2 years ago

Ah ok, that's because the MIN/MIC/MOD type does have support for the integration but it doesn't look like we have added the relevant fields for the batteries to be plumbed through. Therefore can I request that you log a new ticket with the title "Growatt Integration MIN/MIC/MOD type missing Battery details" and then I can get it prioritised. I have a forum post over here where I've prioritised all the issues and I'm working through them in turn: https://community.home-assistant.io/t/growatt-integration-prioritised-list-of-features-for-implementation-fixing/483850 Once you've logged the ticket I'll update that thread with your one as it's only fair I work through things in the correct order.

sidequest-tomas commented 2 years ago

Thanx for quick reply. I posted the ticket : https://github.com/home-assistant/core/issues/82090

but maby it needs more info?

muppet3000 commented 2 years ago

That's all good, I'll add it to the list. Nearer the time I may reach out if you're willing to share your credentials so I can log in as you to the growatt servers and see why the batteries aren't appearing, but I'll do that another time.