Incorrect switching of "controlmode" entity of CW100 thermostat about ems-esp32

[SoftVienka]

On the thermostat, the items are in order (as I wrote above):
External simple
External opt.
Room supply
Room output

In the webUI, the items are displayed in the following order:
(enum_controlmode)
"enum": [ "optimized", "simple", "n/a", "room", "power" ]

However, switching "Control type" in the thermostat is fine.
enum "optimized" change to thermostat "External opt."
enum "simple" change to thermostat "External simple"

(I hope google translated it correctly.)

[proddy]

very clear, love the video! I'll take a look

[MichaelDvP]

For this thermostat there something mixed up in code. It shows the option of RC310 (enum_controlmode1) and used other otions (enum_controlmode) for setting. But both are not identcal to your list. We need to know the numbers of the options you can choose on the thermostat. Please log all and change on the thermostat the mode to all options one after another and post the log and tell the order you have changed.

EMS-ESP32/src/locale_common.h

Lines 323 to 324 in e00eb8e

	MAKE_ENUM(enum_controlmode, FL_(off), FL_(optimized), FL_(simple), FL_(mpc), FL_(room), FL_(power), FL_(constant))
	MAKE_ENUM(enum_controlmode1, FL_(weather_compensated), FL_(outside_basepoint), FL_(na), FL_(room), FL_(power), FL_(constant)) // RC310 1-4

[SoftVienka]

I don't know if I got it right.

We need to know the numbers of the options you can choose on the thermostat.

It can be set on the thermostat as I described above.
In this order:

1. External simple
2. External opt.
3. Room supply
4. Room output

Please log all and change on the thermostat the mode to all options one after another and post the log and tell the order you have changed.

log all.txt

[MichaelDvP]

Thanks, i see the changes at:

T 67: [telegram] Rx: 98 00 FF 00 01 9B 02 03
T 79: [telegram] Rx: 98 00 FF 00 01 9B 01 00
T 109: [telegram] Rx: 98 00 FF 00 01 9B 04 05
T 123: [telegram] Rx: 98 00 FF 00 01 9B 05 04

with order 2, 1, 4, 5. The 3 is not available, but we have an enum, so showing na

Please check ths works: https://github.com/MichaelDvP/EMS-ESP32/releases/tag/latest

[proddy]

that should work - thanks Michael for the fix. Verified with standalone too:

pio run -e standalone -t exec and in console...

test add 0x18 157
call thermostat controlmode “weather compensated” 

[SoftVienka]

Very well.
Now all items are toggled correctly.
The optimizet and simple items are reversed compared to the thermostat.
(not a problem though)

[MichaelDvP]

The optimizet and simple items are reversed compared to the thermostat.

Do you mean the display order or is optimized on the thermostat shown as simple on ems-esp?
Display order is given by the code numbers. 1-optimized, 2-simple, (3-na), 4-room, 5-power
Reordering or skipping the 3 is a lot of extra code.