wills106 / homeassistant-solax-modbus

SolaX Power Modbus custom_component for Home Assistant (Supports some AlphaESS, Growatt, Sofar, Solinteg, Solis, SRNE, Swatten)
317 stars 101 forks source link

[Bug]: Register 0x111 used twice #840

Closed martintothgit closed 6 months ago

martintothgit commented 7 months ago

Describe the bug

The register 0x111 seems to be used twice in code: 1) battery_to_ev_charger 2) disch_cut_off_point_different

In Home Assistant Core Log is following message:

================================== Logger: custom_components.solax_modbus.sensor Quelle: custom_components/solax_modbus/sensor.py:117 Integration: SolaX Inverter Modbus (Dokumentation, Probleme) Erstmals aufgetreten: 17:40:46 (1 Vorkommnisse) Zuletzt protokolliert: 17:40:46

holding register already used: 0x111 disch_cut_off_point_different

Integration Version

2024.04.1

Homeassistant core version

2024.3.3

Inverter brand

SolaX Power

Plugin used

plugin_solax.py

Serial prefix

H34A10

Inverter firmware versions

ARM 1.39 DSP 1.40

Connection Method

SolaX Pocket Wifi 3.0

Dongle firmware

12.01

Detailed Error Log

Logger: custom_components.solax_modbus.sensor Quelle: custom_components/solax_modbus/sensor.py:117 Integration: SolaX Inverter Modbus (Dokumentation, Probleme) Erstmals aufgetreten: 17:40:46 (1 Vorkommnisse) Zuletzt protokolliert: 17:40:46

holding register already used: 0x111 disch_cut_off_point_different

Additional context

No response

nickedw commented 6 months ago

I have the same error message. I don’t know if it’s related, but I’m also seeing a 0 value today for Solax house load, everything else is feeding through correctly. I’m seeing correct values on the Solax cloud app for this parameter.

nickedw commented 6 months ago

I have the same error message. I don’t know if it’s related, but I’m also seeing a 0 value today for Solax house load, everything else is feeding through correctly. I’m seeing correct values on the Solax cloud app for this parameter.

Really sorry, this was me. My external Solax supply breaker had tripped it turns out so the zero reading was correct, the Solax app was reporting what it was supplying to the house from the batteries. Disregard my previous message.

wills106 commented 6 months ago

Just released 2024.04.2 fixing the duplicate register.

martintothgit commented 6 months ago

Thanks a lot for quick fixing! Error message is now gone!