mkaiser / Sungrow-SHx-Inverter-Modbus-Home-Assistant

Sungrow SH Integration for Home Assistant for SH3K6, SH4K6, SH5K-20, SH5K-V13, SH3K6-30, SH4K6-30, SH5K-30, SH3.RS, SH3.6RS, SH4.0RS, SH5.0RS, SH6.0RS, SH5.0RT, SH6.0RT, SH8.0RT, SH10RT, SH5.0RT-20, SH6.0RT-20, SH8.0RT-20, SH10RT-20, SH5.0RT-V112, SH6.0RT-V112, SH8.0RT-V112, SH10RT-V112, SH5.0RT-V122, SH6.0RT-V122, SH8.0RT-V122, SH10RT-V122, SH4.6R
377 stars 95 forks source link

Since FW-Update SAPPHIRE-H_B001.V000.P008-20240430.zip "Status" (Inverter State) has changed from "stop" to "shutdown" #340

Closed Matze-aaa closed 2 months ago

Matze-aaa commented 2 months ago

Before you create an issue, make sure to update to the current version of modbus_sungrow.yaml

Describe the bug:

When the inverter is sent to "shutdown" (eeg via Inverter-Mode Box) the state changed to "stop". Since the FW-Update 20240430 the name of the state has changed from "stop" to "shutdown" (same name as the mode). So the entity "sungrow inverter enable/ stop input selector update" is wrong now. Because its "stop" in the yaml, the inverter starts again after shutdown because of the slider update.

correct: action: input_select.select_option target: entity_id: input_select.set_sg_inverter_run_mode data: option: | {% if is_state('sensor.sungrow_inverter_state', "Shutdown") %} Shutdown {% else %} Enabled {% endif %}

Your Sungrow inverter:

latest version of your yaml.

mkaiser commented 2 months ago

tried to update my inverter, but it looks like the winet-s is having issues.... will take a while for me until I can test & merge

mkaiser commented 2 months ago

What about setting checking here for Shutdown AND Stop?

{% if is_state('sensor.sungrow_inverter_state', "Shutdown") %}

would this work for old and newer firmware versions?

Matze-aaa commented 2 months ago

Good idea, it should be "or", not "and" 😉

mkaiser commented 2 months ago

Just commited - can you double check?

Matze-aaa commented 2 months ago

it looks good, thanks!