[Bug]: Solis - RHI-3P10K-HVES-5G: Setting Engery Control Switch Setting not working

[kai-seipp]

Describe the bug

Hi Guys,

Motivation / goal: Since I now have an electric car I am very much interested in being able to alter some battery storage settings to avoid draining my home battery when chargeing my car. Therefore I now take some interest in these settings.

Issue description: _energy_storage_control_switch: selecting different setting will not work as expected:

• OK: Choosing "Selfuse - No Grid Chargeing" is working as expected.
• Bug: Choosing "Timed Charge/Discharge - No Grid Charging" - is triggering "Timed Charge/Discharge - No Grid Charging" - But is actually allowing grid charging
• Bug: Choosing "Backup/Reserve - No Grid Charging" switches the inverter back to "Selfuse - No Grid Charging"
• Bug / redundant: Choosing "Selfuse" switches the inverter to "Selfuse - No Grid Charging"
• Bug / redundant: Choosing "Timed Charge/Discharge" - is triggering "Timed Charge/Discharge - No Grid Charging"
• unclear: "Battery Awaken" will go back to Selfuse - no Grid Charging. I don't know what the expected behavior would be
• Bug: "Choosing Backup/Reserve - No Timed Charge/Discharge" results in "Selfuse - No Grid Charging"
• Bug: Choosing "Backup/Reserve" as value results the Inverter to be set to "Timed Charge/Discharge - No Grid Charging"

I can't test "Off-Grid Mode" as I don't have my house of grid

Expected: It would be super helpful being able to switch to Backup/Reserve Mode to prevent battery drainage. I would also suggest to strip down the possible options to those we can working.

Enviroment I am Running on 2024.01.8 again, as 2024.2.x kept messing up my statistics (will have to address within an other issue). Running on current HA version

Integration Version

2024.1.8

Homeassistant core version

2024.2.5

Inverter brand

Solis

Plugin used

plugin_solis.py

Serial prefix

110CA2

Connection Method

Waveshare Adapter

Detailed Error Log

There are no errors within the log

Additional context

No response

[wills106]

Once the general issues with 2024.2.x are sorted we can look into specific issues. Hopefully get the connectivity sorted in the next day or two.

[i-jk]

Yes, I'm not sure where all these new values for Energy Control Switch come from? They seemed broadly correct before the recent updates but now there's loads and they don't align with the inverter's actual behaviour.

I'm on 5G RHI 5 kW... All the "good", working ones seem to end in "no grid charging", which is a misnomer.

I'll look at the history and work out when the regressions happened. It's possible we need two lists of settings for different model ranges.

[i-jk]

Ok so I've looked through the history and this commit broke the mode: https://github.com/wills106/homeassistant-solax-modbus/commit/0731c6d8e68ff69109bf79998e2f737cc22ae5ef#diff-b7608f0f1474cba38ac0549ffa6731114128b485fad0fd58c13c165e6b44475bL743

These two options are wrongly labelled now: https://github.com/wills106/homeassistant-solax-modbus/blob/9de88e1a614c7b14e81f4dd0b2e3e4c27c68977a/custom_components/solax_modbus/plugin_solis.py#L775

The issue is this will make people's inverters discharge when they think they're choosing self use. It's bad, it will potentially cost people money - I just exported all my stored solar due to this mislabeled option and now need to pay for electricity for tonight.

I think we need more rigor around commits that change these.

And we also need to look at the naming because it's pretty crazy now, some names are so long they can't be read on mobile.

[i-jk]

The correct values for line 775-776, and 1350-1351 of plugin_solis.py are:

                33: "Self-Use",
                35: "Timed Charge/Discharge",

Sorry, can't raise PR right now @wills106, on holiday doing this from mobile. It's a critical regression to fix though 🙏

[wills106]

Does "Energy Storage Control Switch" still need correcting?

[kai-seipp]

Does "Energy Storage Control Switch" still need correcting?

Hi Will,

I did install the latest Version (did run on January version until yesterday, as I had major issues with newer once) to check. Yes there had been no changes made regarding this issue. So yes, that would be great!

But I will also open a new issue to make the integration working with Solis again in general. Currently the integration can't be used when an inverter goes offline during nighttime. (Just as heads up).

[neilrfuller]

Hi Folks,

Think I am experiencing these issues too. Very much a newbie with all this, but tried to switch inverters to "Backup/Reserve No Grid Charging". Inverters immediately started to charge from grid. I'm on Intelligent Octopus go, my motivation was to leave my batteries set to not charge from grid (I have sufficient solar to do that these days), but prevent them from being drained overnight when my car starts smartcharging. Be great if this could be fixed, everything else seems to work great, my thanks and admiration for all your efforts here.

[i-jk]

This integration is broken for Solis, @wills106. Can we get a revert of the problematic commits referenced in this issue post? Then all will be well.

[i-jk]

How can we mark a bug as critical? This one is

[wills106]

This integration is broken for Solis

As in completely doesn't work? Or the incorrectly named mode?

Do you agree with the comment? https://github.com/wills106/homeassistant-solax-modbus/commit/0731c6d8e68ff69109bf79998e2f737cc22ae5ef#commitcomment-143902523