-
-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
thermostat mode 'manual' missing as option #932
Comments
that's true, we don't have a "Manual" option for any of the thermostats other than the RC100, RC300, RC310. Not sure why. @MichaelDvP ? |
I think because most of the thermostats dont have a manual mode. |
I will also take a look at mentioned issues to see if I can figure something out. |
You can use terminal, |
Did some tests, will look in more detail later today. Just wanted to share the logfile. Did see this error message when pressing the hand icon: |
With log debug it's very nasty to search only the watch output. Try https://github.com/MichaelDvP/EMS-ESP32/releases/tag/latest Seems the mode filed is not affected, so we can't add it to mode setting. |
Thanks for the quick addition I will give the latest version a try and let you know,. |
Works as expected. Takes around 10 - 20 seconds before both values (manualtemp and selected room temp) are equal. Just something to pay attention to when testing. Do you want/need more logs when pressing buttons doing actions on the Moduline 400? |
I'm thinking how to use the mode command for night/day/auto/manual.
The seltemp is taken from the reporting telegram from thermostat, show what is really reported. This telegram is send every 10 sec, and thermostat also needs some time to set the new value and report it. This monitor telegram 0x41 is not readable, in the log it is always , so we have to wait until the thermostat send it. I think we should change the readback to the setting telegram. |
Yes, tempautotemp is in normal operation always zero, only if you are in automode and change the temperature it is set to this value and is cleared at next scheduled change. But i want to know how we can change the mode from ems-esp.
Same in the other direction:
|
I'll do the same test too on my RC20 which also has the manual mode |
Thanks for the hint, RC20 with setting 0xA8 have modes off/manual/auto and mostly the same structure as RC30 with 0xA7. I think the modes (night/day/auto) for RC30 are wrong, this should also be off/manual/auto. When pressing the manual button the mode switches to 1. There is no need for this crude logic checking the temperature fields, only the mode enum needs to be adapted. Seltemp for RC20 shoud work with manual an auto. |
Seems the RC20/RC20_N modes are a little bit mixed for reading and setting. I've tried to clean up, but i'm not sure which modes these thermostats really have. In https://github.com/MichaelDvP/EMS-ESP32/releases i have now set RC20/RC30 to modes |
Thanks for the adjustments. Looks better now. Thermostat has 3 buttons, auto, manual, off (and a fourth to show info in the display) I did some tests (if not mentioned the operation was done from ems-esp); send hc1 mode == auto pressing the auto button on the thermostat final test to set it to auto from ems-esp Attached the full log, don't know yet why it didn't go in to auto mode on the first attempt |
Thx, have you checked the seltemp command in manual and auto mode? |
@MichaelDvP , i forgot to do a deep test with that. I did notice the adaptation of the temperature to what would be expected. deep test is now also done, see logfile test sequence from what I see all works as it should (same behavior as when its set directly on the thermostat itself) |
I was not able to react this weekend so I'm a little late with my reaction. Sorry for that.
When it is in Off mode. the used temperature will come from telegram A7 offset 24. I'm running version 3.4.4 from a few months ago and got an error uploading your dev version. So I probably missed some important prerequisites? I will have a look at that and see if I can test your version also. |
I tested your build @MichaelDvP in my RC20 and it works fine, all configurations (manually on the thermostat and via EMS-ESP). On a few occasions using the thermostat the command didn't come through (incomplete telegram warning) but most times it worked first time. |
@kwertie01 Thanks, i'll add for seltemp the offset24 in mode:off. |
I've updated to dev-17 with changelog: Off-mode and temperature set by seltemp. Also cleaned up a bit and moved the set_temp functions to inline (saves ~0,5k flash). Made a PR. |
Thank you @MichaelDvP for the information about the partition change. |
You can just set seltemp to 0, this sends to offset 28 if in auto mode. |
I think this is solved and merged in the official release, closing. Please reopen if there is anything missing. |
Bug description
thermostat (hc1 in my case) is missing the 'manual' mode
Steps to reproduce
put thermostat in manual mode (on Moduline 400 - press hand icon)
Adjust the heat index (number.thermostat_hc1_selected_room_temperature) in HA
temperature is adjusted and heater starts
adjust temperature on thermostat
heating isn't affected, burner state remains the same
put thermostat in manual mode
adjust temperature once more as previous adjustment is not used
heating is now adjusted to new temperature
(heating either starts or stops depending on the room- and selected temp)
Expected behavior
thermostat should remain in manual mode and accept new temp setting from both HA and manual operation
Device information
{
"System Status": {
"version": "3.5.0b7",
"platform": "ESP32",
"uptime": "000+05:26:35.359",
"freemem": 118,
"free_app": 74,
"reset reason": "Software reset CPU / Software reset CPU"
},
"Network Status": {
"connection": "Ethernet",
"hostname": "ems-esp",
"MAC": "0C:B8:15:4A:EA:0F",
"IPv4 address": "*****",
"IPv4 gateway": "****",
"IPv4 nameserver": "208.67.222.222",
"phy type": 1,
"eth power": 16,
"eth phy addr": 1,
"eth clock mode": 0,
"static ip config": false,
"enable IPv6": false,
"low bandwidth": false,
"disable sleep": false,
"AP provision mode": "disconnected",
"AP security": "wpa2",
"AP ssid": "ems-esp"
},
"NTP Status": {
"network time": "disconnected",
"enabled": false,
"server": "time.google.com",
"tz label": "Europe/Amsterdam"
},
"OTA Status": {
"enabled": true,
"port": 8266
},
"MQTT Status": {
"MQTT status": "connected",
"MQTT publishes": 6446,
"MQTT queued": 0,
"MQTT publish fails": 0,
"MQTT connects": 1,
"enabled": true,
"client_id": "ems-esp",
"keep alive": 60,
"clean session": false,
"base": "ems-esp",
"discovery prefix": "homeassistant",
"nested format": 1,
"ha enabled": true,
"mqtt qos": 0,
"mqtt retain": false,
"publish time boiler": 10,
"publish time thermostat": 10,
"publish time solar": 10,
"publish time mixer": 10,
"publish time other": 10,
"publish time sensor": 10,
"publish single": false,
"publish2command": false,
"send response": false
},
"Syslog Status": {
"enabled": false
},
"Sensor Status": {
"temperature sensors": 0,
"temperature sensor reads": 0,
"temperature sensor fails": 0,
"analog sensors": 0,
"analog sensor reads": 0,
"analog sensor fails": 0
},
"API Status": {
"API calls": 0,
"API fails": 0
},
"Bus Status": {
"bus status": "connected, tx issues - try a different Tx Mode",
"bus protocol": "Buderus",
"bus telegrams received (rx)": 15405,
"bus reads (tx)": 3031,
"bus writes (tx)": 0,
"bus incomplete telegrams": 0,
"bus reads failed": 559,
"bus writes failed": 0,
"bus rx line quality": 100,
"bus tx line quality": 85
},
"Settings": {
"board profile": "E32",
"locale": "en",
"tx mode": 1,
"ems bus id": 11,
"shower timer": false,
"shower alert": false,
"rx gpio": 5,
"tx gpio": 17,
"dallas gpio": 4,
"pbutton gpio": 33,
"led gpio": 2,
"hide led": false,
"notoken api": false,
"readonly mode": false,
"fahrenheit": false,
"dallas parasite": false,
"bool format": 1,
"bool dashboard": 1,
"enum format": 1,
"analog enabled": true,
"telnet enabled": true
},
"Devices": [
{
"type": "Boiler",
"name": "GBx72/Trendline/Cerapur/Greenstar Si/27i",
"device id": "0x08",
"product id": 123,
"version": "06.07",
"entities": 67,
"handlers received": "0x10 0x15 0x1C 0x18 0x19 0x34 0x2A",
"handlers fetched": "0x16 0x33 0x26",
"handlers pending": "0xBF 0x11 0xC2 0x14 0x1A 0x35 0xD1 0xE3 0xE4 0xE5 0xE6 0xE9 0xEA"
},
{
"type": "Thermostat",
"name": "Moduline 400",
"device id": "0x10",
"product id": 78,
"version": "03.07",
"entities": 32,
"handlers received": "0xA3 0x06 0xA2",
"handlers fetched": "0x41 0xA7 0x40 0x3F 0x3A",
"handlers pending": "0x12 0x13 0x38 0x39",
"handlers ignored": "0x1A 0x35"
},
{
"type": "Controller",
"name": "BC10",
"device id": "0x09",
"product id": 190,
"version": "01.04",
"entities": 0
}
]
}
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: