mletenay / home-assistant-goodwe-inverter

Experimental version of Home Assistant integration for Goodwe solar inverters
https://discord.gg/TaXyWXT
MIT License
146 stars 38 forks source link

PV1 Voltage = PV2 Voltage and PV2-Currunt alway 0A @ 3-Stringed GW25ET #245

Closed hbpv closed 8 months ago

hbpv commented 8 months ago

Hi

We've two major Problems with values:

  1. PV1 Voltage is alway the same as PV2 Voltage, but the Strings are strongly different. Please see the screenshot. Is it possible that there is an copy-paste mistake in polling the same values inside the script?

  2. PV2 Current stay always at 0. As this String is producing >3kwh, this is wrong.

Thanks and best, Max

Pv2-current-always-0 Pv1-voltage=Pv2-voltage

mletenay commented 8 months ago

I believe this is correct. The ETT inverters have 2 string per MPPT and the "pv" sensors represent strings. So pv1 and pv2 are using same mppt - thus same voltage (and current on 2nd is probably the same, just not reported by inverter). You have to look also at the pmppt and imppt.

hbpv commented 8 months ago

Okay, so the entities we get are wrong? The Inverter has 6 PV Ports 1+2 = PV1 3+4 = PV2 5+6 = PV3

Our first String is on Input 1 (PV1-1) Our second String is on Input 3 (PV2-1) Our third String is on Input 5 (PV3-1)

In the home assistant integration, the way your argument, we should get the entities PV1 + PV3 + PV5. But it only shows 1,2,3. Also, when they have the same Voltage, the other values should be the same regarding your idea, but they're not.

Please see attached the Screenshot for the integration.

image image

hbpv commented 8 months ago

Here you’ve got the sems values, where it shows correctly 3 different strings and their voltages etc

IMG_3757

mletenay commented 8 months ago

Well, what I know is there is no copy/paste mistake in the code - see for yourself: https://github.com/marcelblijleven/goodwe/blob/master/goodwe/et.py The rest is responsibility of inverter what values it provides.

mletenay commented 8 months ago

... and the current integration does not have the latest library code yet, so you may not see the pv5+ sensors. If you are able, try the library directly and run inverter_check.py to see all values inverter reports.

hbpv commented 8 months ago

Okay.. And now? How could I further help to get this resolved?

As you can see in the SEMS Screenshot, there are 3 different values. So maybe we should check, why the integration/inventer is not resolving the ports that are connected (PV1-1 = PV1) + (PV2-1 = PV3) + (PV3-1 = PV5). Maybe wrong pull of modbus registers? In the et.py i see a lot more possible PV's up to 16.

Can i somewhere check/log which values are provided? Maybe we can there find the problem.

here is the other solargo screenshot! 6PVS

BUT: The integration only fetches (at least it looks like) PV1/2/3 but schould PV1/PV3/PV5 as 2, 4, 6 are the parell, not used ports, of course with same Voltage as you said. So

IMG_3758

hbpv commented 8 months ago

... and the current integration does not have the latest library code yet, so you may not see the pv5+ sensors. If you are able, try the library directly and run inverter_check.py to see all values inverter reports.

Okay.. and how to update our library? Could give me a short how to? Thanks

mletenay commented 8 months ago

I will release next version of the library probably over the weekend and then adjust the HA integration as well.

hbpv commented 8 months ago

I will release next version of the library probably over the weekend and then adjust the HA integration as well.

Okay.. i just installed the moist recent version of hacs-goodwe from you, but problem persists. Is this the same release channel?

hbpv commented 8 months ago

For your release, please be aware that with hacs integration:

MPTT entities are working fine (all 3 with correct values) BUT MPPTX Current are W(Watt) instead of A(ampere), see screenshot image

And:

As i showed you before, all (exept PV4 is missing completly??) PV's are showing, but with wrong Voltages:

Starting with

(MPPT1) PV1 - Voltage and Current OK (MPPT1) PV2 - Voltage OK while same as MPPT1-PV1, but Current and Power should be 0 (MPPT2) PV3 - Voltage and Current OK (MPPT2) PV4 - completely missing, should be Power 0, and Currunt/Coltage like PV3 (MPPT3) PV5 - Voltage and Current OK (MPPT3) PV6 - Voltage OK while same as MPPT3-PV5, but Current and Power should be 0

image