home-assistant-goodwe-inverter icon indicating copy to clipboard operation
home-assistant-goodwe-inverter copied to clipboard

Today's PV Generation not matches Sems Portal

Open kolossboss opened this issue 3 years ago • 11 comments

Hi, The sensor sensor.goodwe_e_day did not match the PV Energy Sensor from Sems Portal.

  • This integration shows 9 kwh
  • Sems integration 14,2 kwh (correct one)

A few days ago this was working correct.

Inverter: GW3648-EM

kolossboss avatar Sep 05 '21 13:09 kolossboss

That's weird. I have checked the code and there were no changes related to e_day sensor recently. And it is also covered by unit tests for some time already. Can you observe it for several days ? I can't find a problem there.

mletenay avatar Sep 05 '21 14:09 mletenay

Yes very weird. The connection is also very unreliable, but I dont think thats the reason of the gap. There is also a small gab in "Total Generation" and the graph looks also different. (with this integration the graph still rises even though it's night ... it's confusing)

Bildschirmfoto 2021-09-05 um 20 31 43 Bildschirmfoto 2021-09-05 um 20 31 36 Bildschirmfoto 2021-09-05 um 20 31 19 Bildschirmfoto 2021-09-05 um 20 31 06

kolossboss avatar Sep 05 '21 18:09 kolossboss

The update to v0.8.9 fixes the unreliable connection issue. Thanks a lot!

But the differences between this integration and SEMS portal still exists.

kolossboss avatar Sep 06 '21 12:09 kolossboss

Is the discrepancy between this and SEMS still there ? Couldn't it be we or them just use wrong sensor name for the values ? Isn't there some other sensor which +/- matches the SEMS values ?

mletenay avatar Sep 06 '21 22:09 mletenay

Yes the discrepancy is sill there.

Its possible that anything is wrong, but I'm not sure what. The graph looks also very weird.... There is also a discrepancy in the "Total Generation Sensor" (last two screens)

From this Integration: image image

From "Goodwe Sems API" Integration: image image

From "Goodwe SEMS API to MQTT sensor" Integration: image image

Total Generation Sensor: image image

kolossboss avatar Sep 07 '21 05:09 kolossboss

Hi, Not sure if I understand precisely what you mean with PC Energy Sensor but maybe this something to investigate and see if your graphs make more sense. With the ES Inverter, sensor.goodwe_e_day combines PV consumption with battery consumption. In other words, SEMS Portal show only PV generation where sensor.goodwe_e_day shows PV consumption + Battery consumption. For the ES inverter sensor.goodwe_ppv match up to what I see in the SEMS Portal.

GIJOE-SA avatar Sep 18 '21 06:09 GIJOE-SA

Hi, its a typo, I mean PV Energy Sensor. Ok, its possible that sensor.goodwe_eday combines in any way the battery. But I have a 7kwh Battery. I think that there must be a much larger delta between this 2 sensors. But its only 50kwh and 2800kwh in total.

Sensor.goodwe.ppv only shows W and nicht kwh. But the sensor is also working correct for me. I used the sensor in the new Energy Management portal.

kolossboss avatar Sep 20 '21 09:09 kolossboss

The way I understand it is: SEMS portal - Load Consumption = PV Output + Grid Consumption. sensor.goodwe_e_day = combined consumption from PV and battery in that point in time sensor.goodwe_e_load_day combined consumption from GRID, PV and battery in that point in time. So I do not think you can compare these two sensors with SEMS Portal consumption values

GIJOE-SA avatar Oct 26 '21 19:10 GIJOE-SA

I have the same invertor (GW3648-EM) and I see similar behavior. The sensor sensor.today_s_pv_generation does not match the actual pv generation. If I sum the PV power over time I get a value close to the SEMS portal one. This sensor.today_s_pv_generation is something different.

Jayksopp avatar Jan 09 '22 12:01 Jayksopp

I have the same issue, is there some final solution to the problem ?

matejgordon avatar Jan 31 '22 12:01 matejgordon

Is there any fix for this? I have the same problem. Today's Load is also wrong. Plugin shows: 6,7kwh, SEMS shows: 6,76(self-usage)+2,64(bought from grid)= 9,4kw Zm7dsDaTjG

nazeg avatar Feb 12 '22 15:02 nazeg

We have the same situation with Goodwe GW3048: there is a big difference between SEMS and Modbus values. - We haven't connected a battery. Anybody has found a solution? Thank you

smileyman-1 avatar Mar 12 '23 19:03 smileyman-1

Same issue here. Inverter GW3648D-ES installed last week. The "sensor.total_pv_generation" and the "sensor.today_s_pv_generation" values exported by the inverter through this integration show solar production even at night, where the instantaneous power produced by the panels is obviously zero.

This is weird. Even if the reason of this behaviour is that those sensors show the combined solar+battery consumption, despite the sensors' names suggesting otherwise, the produced data is pretty useless, in my opinion.

The only quick fix I see is to define some helper (virtual) sensors that integrate the different instantaneous powers to calculate the energies, and some counters to add up these values in a daily/monthly/yearly basis.

orestesmas avatar Mar 17 '23 18:03 orestesmas

@orestesmas It can hardly be related to the battery consumption, since I don't have a battery connected at the moment.

smileyman-1 avatar Mar 17 '23 18:03 smileyman-1

Hi, this maybe something related to your model inverter, on my GW5048D-ES both these sensors in Home Assistant match the mobile app data 100%. image

GIJOE-SA avatar Mar 22 '23 08:03 GIJOE-SA

Did some digging, on the first screen there is "Generation Today" which seems higher than the value of "Generation Today" when you select the inverter itself. On this stage it seems that that inverter value "Today's PV Generation sensor" is lacking behind a couple of hours. Maybe the reason why I did not use "Today's PV Generation" sensor in my Energy dashboard. I have created the following in my config file which I use for daily and monthly (Energy dashboard)

'- platform: integration source: sensor.pv_power name: pv_generation_sum round: 1 unit_prefix: k method: left'

image

and

'utility_meter: pv_gen_daily: source: sensor.pv_generation_sum cycle: daily pv_gen_monthly: source: sensor.pv_generation_sum cycle: monthly'

image

Hope it helps.

GIJOE-SA avatar Mar 22 '23 11:03 GIJOE-SA

Are you guys running batteries? Energy generation

Same issue here. Inverter GW3648D-ES installed last week. The "sensor.total_pv_generation" and the "sensor.today_s_pv_generation" values exported by the inverter through this integration show solar production even at night, where the instantaneous power produced by the panels is obviously zero.

This is weird. Even if the reason of this behaviour is that those sensors show the combined solar+battery consumption, despite the sensors' names suggesting otherwise, the produced data is pretty useless, in my opinion.

The only quick fix I see is to define some helper (virtual) sensors that integrate the different instantaneous powers to calculate the energies, and some counters to add up these values in a daily/monthly/yearly basis.

Thats how the logic works for GoodWe with battery. Any solar that goes into the battery doesn't count. Any energy that comes out of battery is added to the generation counter. Battery is defined as a solar storage, there is no actual battery storage. Its a valid implementation if looked on it own. If looked when added to home assistant its not correct. Only alternative is to measure the energy with the actual power instead of using energy generation given by GoodWe.

midadami avatar Apr 21 '23 06:04 midadami