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]: Unknown solax_run_mode on newest firmware SOLAX X1-Hybrid-G4 #681

Closed ivankarpov closed 9 months ago

ivankarpov commented 10 months ago

Describe the bug

I recently updated the firmware of the inverter to the latest version. Reported by the intergration: ARM - 1.38 DSP - 1.41

After the update the running mode of the inverter while using remote power control is reported as unknown. On the screen of the inverter is Normal(R) displayed

0-02-05-25df13860360382e7c8e3d378655a7a76b973cec70772dc8031fb44ad24f4ae1_273c048b560 Screenshot 2023-12-28 at 10 23 48

Integration Version

2023.12.5

Homeassistant core version

2023.12.3

Inverter brand

Solax

Plugin used

plugin_solax.py

Serial prefix

H475

Inverter firmware versions

ARM - 1.38 , DSP - 1.41

Connection Method

SolaX Poket WiFi 3.0

Dongle firmware

12.01

Detailed Error Log

No response

Additional context

No response

wills106 commented 10 months ago

If you either comment out or delete Lines 3528 - 35538

https://github.com/wills106/homeassistant-solax-modbus/blob/7aeb55b54dde4bd29849fd9a30c9b518183b335b/custom_components/solax_modbus/plugin_solax.py#L3524-L3542

So you just have:

    SolaXModbusSensorEntityDescription(
        name = "Run Mode",
        key = "run_mode",
        register = 0x9,
        register_type = REG_INPUT,
        allowedtypes = GEN4,
        icon = "mdi:run",
    ),

What does it return when in Normal(R) mode? As the docs don't mention that mode, so I don't know what to map it to.

matomatusov commented 10 months ago

Hi @Wills106 normal (R) is when it is running modbus remote control.

wills106 commented 10 months ago

I meant what does the Integration return with the modified "Run Mode" with the scaling removed. It will return a number.

ivankarpov commented 10 months ago

Hi @wills106

The "Normal (R)" Mode is 20 :), do you need a marge request or you will add it ?

Regads

wills106 commented 10 months ago

I'll add it in thanks.

wills106 commented 10 months ago

This should be corrected in 2024.01.2

wills106 commented 9 months ago

@ivankarpov can this issue be closed down now?

ivankarpov commented 9 months ago

Yes, it is displayed as expected with the new version of the integration. Thank you1