By design or ... ?

[c0mplex1]

If you switch off all inverters at once switch.ecu_{ECU-ID}_all_inverters_on_off, the power generation stops, but the on/off switch position of the inverters does not change.

If you switch off a single inverter switch.inverter_{Inverter-ID}_on_off and then switch off all inverters at once switch.ecu_{ECU-ID}_all_inverters_on_off, after which you switch all inverters on again, all inverters will start working again, including the one that was just switched off. However, the on/off switch of the inverter in question will remain off.

[HAEdwin]

Good point, thanks for noticing this - maybe that's why the ECU UI does not show the status unless you change anything

The first case is weird, I can't reproduce it. The switch turns off and keeps off, even on non compatible ECU's the switch works but should not because it is incompatible. So that's a fix I need to do.

The second case I can fix using force states, but keeping it in sync with the true state is important.

I will look into it later

[c0mplex1]

I'm talking about the HA integration, not about the ECU UI.
The ECU UI follow the settings just fine.

[c0mplex1]

[HAEdwin]

I understand the issue.
I've looked into it and concluded that the switches indeed need a overhaul.
I'll be gone for the weekend (Koningsdag) so will spend time on it later when sober again :)

[c0mplex1]

You're doing that right.
The bow can't always be tense.
Happy King's Day.

[HAEdwin]

I'm thinking how to best represent the correct state of the switches and improve interpretation, so a truth table with two inverters as an example would help to discuss the matter:

All Inverters On/Off	Inverter 1 On/Off	Inverter 2 On/Off
0	0	0
0	0	1
0	1	0
1	1	1

The "All Inverters On/Off" switch currently is a debatable switch label because how do you interpret it when it is off? Are all inverters off? (which does not have to be true). I think the best solution here is to rename the switch "All Inverters On" then:

• Changing the status of an Inverter On/Off switch means going through all inverter statuses and if necessary changing the status of the "All Inverters On" switch.
• Conversely, changing the "All Inverters On" switch means going through and changing all inverter On/Off switches status.

Alternative names: "All Inverters Enabled", "Enable All Inverters", "All Inverters Activation Status", "All Inverters Status", "All inverters forced off" (like @dewi-ny-je suggests)
I would like to hear your opinions. I would personally go for the "All Inverters On".

Last option and easiest way out is to remove the All Inverters On/Off switch which I introduced in v2.6.0

[HAEdwin]

I absolutely agree with you, it is too confusing now. You can build everything in but then you are ignoring the possibilities and flexibility of Home Assistant itself. The switches still need to be improved and that requires quite a few adjustments. My plan is to disable entities that are not compatible with the ECU model. So the switch.inverter_{Inverter-ID}_on_off will be disabled for an ECU-B for example.

[c0mplex1]

What do you mean with "The switches still need to be improved"?
For the ECU-R Pro, if the automation sends the command switch.turn_off InverterX, then InverterX go Off and the switch in the InverterX device configuration UI goes to the Off position too.
When the automation sends the command switch.turn_on, the Inverter goes On and the switch in the device configuration UI goes On too.
I don't see any issues here.

[dewi-ny-je]

I have no preference about maintenance or removal of the general switch, but the switches on the individual inverters should be surely renamed to "disable inverter" which conveys the correct meaning.

[c0mplex1]

I think "disable inverter" is a sign that you can no longer operate the inverter.
While "switch off" the inverter means that it can no longer generate energy. (in this case)
So "Switch Off" is better.

[HAEdwin]

Improvement of the switches is purely code related. Disable indeed is more suited for situations where you don't use it anymore and hide it from visibility in the UI, disable also implies that you are unable to use it in automations. Here's the style guide: https://design.home-assistant.io/#components/ha-switch

[dewi-ny-je]

I have to say that the switches on the inverters are actually fine: I thought they were misleading and inverted because they were always on "off" and the labels were working, but after I used for the first time the global enable/disable I found that flipping the global one back to off the panels stayed off, because the inverters switches were on "off" by default.
Also, each inverter power limit was set to 0!!

Now I enabled each inverter on/off and manually set the power limit to something reasonable and the power is back.

So only the global one is misleading, but as stated by others, it could simply be omitted and that's it. Switch groups can be used instead

[HAEdwin]

In the new release it will be removed as well as incompatible entities (i.e. the inverter switch if having an ECU-B as parent).

[HAEdwin]

Solved in release 2.8.0