search

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)
285 stars 85 forks source link

[Bug]: Sofar - 3.3-12KTLX-G3 PV Power Total no longer correct after firmware update of inverter #784

Open darkrain-nl opened 4 months ago

darkrain-nl commented 4 months ago

Describe the bug

After updating my inverter to firmware version: V100011-10-14 (sorry don't know the old firmware version) the PV Power Total value is changed from scale 0.01 to 0.1 (like the hybrid models). Another change I noticed is it doesn't show 10-90W values anymore, only 100W and above, so lower then 100W is shown as 0W. afbeelding

The scale I can fix locally, and I could do a pull request for it, but that would probably be a breaking change on inverters that are still on the older firmware... Or is there a way to have it be depending on firmware?

What are the options here, is there something that can be done to show the 10-90W values as well, can the scale be adjusted just for this firmware version?

Integration Version

2024.02.6

Homeassistant core version

2024.2.5

Inverter brand

Sofar

Plugin used

plugin_sofar.py

Serial prefix

SS2ES1

Connection Method

Waveshare adapter

Detailed Error Log

No response

Additional context

No response

wills106 commented 4 months ago

I have seen your PR.

I think we either need to detect Firmware version through the Integration. Or have the option to manually override the scaling.

darkrain-nl commented 4 months ago

That makes sense, I do not have time to work on this anytime soon... but feel free to close the current PR.

Can come back to it later, or somebody else might have a go...

wills106 commented 4 months ago

I'll leave it open as a reminder.

We have done similar overrides on SolaX. So I can't see it being a big issue.