Add cooling mode to Bosch room thermostat II (BTH-RM) #7681
Conversation
mattreim
commented
Mar 26, 2024
mattreim
commented
mattreim
changed the title
|
Tricky, I don't have any experience with that device, but it looks like the PR introduces a breaking change due the renamed values? Is the removal of "auto" and "heat" really needed or is it better to only extend the existing modes with the new ones? |
|
I think renaming "auto" is not necessary, but heat in manual is more understandable here. Incorrect operation could occur if "heat" is used twice. operation manual:
|
|
The modes auto, manual and off are functional in both channels cool and heat. |
|
Unfortunately, when you turn off the device, the following error message appears:
And this under cluster info:
|
devices/bosch/room_thermostat2.json
Outdated
| @@ -28,7 +28,7 @@ | |||
| }, | |||
| "meta": { | |||
| "values": { | |||
| "config/mode": {"auto": 0, "heat": 1, "off": 5} | |||
| "config/mode": {"auto": 0, "manual": 1, "off": 5} | |||
There was a problem hiding this comment.
Please do not change the names here. Those are aligned with the respective zigbee attribute and REST API clients do not expect different values. We should stay consistent here.
devices/bosch/room_thermostat2.json
Outdated
| @@ -148,19 +178,19 @@ | |||
| "cl": "0x0201", | |||
| "dt": "0x30", | |||
| "ep": 1, | |||
| "eval": "if (Item.val == 'auto') { 0 } else if (Item.val == 'heat') { 1 } else if (Item.val == 'off') { 5 };", | |||
| "eval": "if (Item.val == 'auto') { 0 } else if (Item.val == 'manual') { 1 } else if (Item.val == 'off') { 5 };", | |||
devices/bosch/room_thermostat2.json
Outdated
| { | ||
| "name": "config/devicemode", | ||
| "refresh.interval": 3600, | ||
| "read": { | ||
| "at": "0x001C", | ||
| "cl": "0x0201", | ||
| "ep": 1, | ||
| "fn": "zcl:attr" | ||
| }, | ||
| "write": { | ||
| "at": "0x001C", | ||
| "cl": "0x0201", | ||
| "dt": "0x30", | ||
| "ep": 1, | ||
| "eval": "if (Item.val == 'off') { 0 } else if (Item.val == 'cool') { 3 } else if (Item.val == 'heat') { 4 };", | ||
| "fn": "zcl:attr" | ||
| }, | ||
| "parse": { | ||
| "at": "0x001C", | ||
| "cl": "0x0201", | ||
| "ep": 1, | ||
| "eval": "if (Attr.val == 0) { Item.val = 'off' } else if (Attr.val == 3) { Item.val = 'cool' } else if (Attr.val == 4) { Item.val = 'heat' };", | ||
| "fn": "zcl:attr" | ||
| }, | ||
| "default": "heat" | ||
| }, |
There was a problem hiding this comment.
Is this confirmed to be working? Bosch already seems to use a manufacturer specific attribute to set the heating modes and apparently disregards the standard attribute.
There was a problem hiding this comment.
At least it works for me. The two channels "heat/cool" then also use the normal modes: auto, manual(heat) and off.
The attribute “devicemode” was also obvious to me.
There was a problem hiding this comment.
But what do I do here with the double heat and the off(450) problem?
There was a problem hiding this comment.
How about if I use mode/preset instead of devicemode/mode?
mode --> heat, cool and off
preset --> manual, auto and holiday
or
mode --> heat, cool and off
devicemode --> manual, auto and off
devices/bosch/room_thermostat2.json
Outdated
| "eval": "if (Attr.val == 0) { Item.val = 'auto' } else if (Attr.val == 1) { Item.val = 'manual' } else if (Attr.val == 5) { Item.val = 'off' };", | ||
| "fn": "zcl:attr", | ||
| "mf": "0x1209" | ||
| }, | ||
| "default": "heat" | ||
| "default": "manual" |
|
I'm sorry, but I cannot say that the recent changes can get my vote. My perception is that thing got even more complicated and drift away from common user experience. Is there any particular reason why you chose to ignore the requested changes? If I haven't overlooked anything, the amendments you've made over time state that all the required mode changes can be done through attribute 0x001C, being the zigbee intended way. That makes we wonder why not to use it? |
|
Sorry, maybe I didn't make myself clear. Here are the device attributes: 0x001C:
0x4007:
|
|
The different states: (Mode --> devicemodes) heat --> manual, auto and off cool --> manual, auto and off off --> off |
|
They're just names and it works like that. |
|
Inspired by Home Assistant;
|
