wills106 / homeassistant-solax-modbus

SolaX Power Modbus custom_component for Home Assistant (Supports some Ginlong Solis, Growatt, Sofar Solar, TIGO TSI & Qcells Q.Volt Hyb)
288 stars 85 forks source link

[Bug]: value of sensor.inverter_power in the same as inverter_power_l1 but not l1+l2+l3 #667

Open gregorwolf1973 opened 7 months ago

gregorwolf1973 commented 7 months ago

Describe the bug

The value of the sensor inverter_power is the same as inverter_power1 but it should be inverter_power1 + inverter_power2 + inverter_power3. power I think this also has impact on other values but i'm not sure.

Integration Version

2023.12.4

Homeassistant core version

2023.12.2

Inverter brand

SolaX X3 Hybrid

Plugin used

plugin_solax.py

Serial prefix

H3PE05

Inverter firmware versions

210117LC

Connection Method

RJ45 TCP/IP Network

Dongle firmware

none

Detailed Error Log

No response

Additional context

No response

martin-klima commented 5 months ago

I just checked the sensors in my SolaX X3 Hybrid G3 installation and everything looks good.

image image

gregorwolf1973 commented 5 months ago

Thanks! I checked several times and reinstalled, rebooted. I have a X3 Hybrid G1/2 and here it is different. Zwischenablage01

wills106 commented 5 months ago

Your Inverter is a Gen3 https://github.com/wills106/homeassistant-solax-modbus/blob/a558ca3999ec6518d09e2cae932400e1f05003d7/custom_components/solax_modbus/plugin_solax.py#L6470 For some odd reason SolaX labeled some up as X3 Hybrid G1/2, they share the same register set as the X1 Gen3.

I don't believe there was a true 3Phase equivalent of the Gen2 X1 SK-SU / SK-TL hybrids.

There could possibly be a bug in the Firmware on your Inverter or a hardware change as there are 5 known X3 Gen3's variations. You may need a template sensor to build an Inverter Power out of L1-L3 combined.