Domochip / WPalaControl

D1Mini to replace Palazzetti, Jotul, TurboFonte, Godin stove ConnectionBox
42 stars 9 forks source link

Set silent mode returns a timeout error #29

Closed Flobul closed 8 months ago

Flobul commented 9 months ago

Hi.

When SET+SLNT+1 command is sent, the room fan stops, but the cmd returns a timeout issue. When SET+SLNT+0 command is sent, nothing happens (is it supposed to recover last fan value ?)

Mac@Flobul % curl "http://192.168.XXX.XXX/cgi-bin/sendmsg.lua?cmd=SET+SLNT+0"
{"INFO":{"CMD":"SET SLNT","RSP":"OK"},"DATA":{"SLNT":0,"PWR":1,"F2L":0,"F2LF":0},"SUCCESS":true}%    

Mac@Flobul % curl "http://192.168.XXX.XXX/cgi-bin/sendmsg.lua?cmd=SET+SLNT+1"
{"INFO":{"CMD":"SET SLNT 1","RSP":"TIMEOUT","MSG":"Stove communication failed"},"DATA":{"NODATA":true},"SUCCESS":false}%   
Domochip commented 8 months ago

Hi,

SET+SLNT+1 should not return a timeout error, this is effectively a bug. Regarding SET+SLNT+0, it does nothing...

I'll have a look at this bug shortly. thx

Domochip commented 8 months ago

Hi, Could you give it a try with the latest pre-release? If the error still occur, I'll need your GET+STDT to understand, what happens.

Flobul commented 8 months ago

Hi.

Mac@Flobul % curl "http://192.168.23.49/cgi-bin/sendmsg.lua?cmd=SET+SLNT+0"
{"INFO":{"CMD":"SET SLNT","RSP":"OK"},"DATA":{"SLNT":0,"PWR":1,"F2L":0,"F2LF":0},"SUCCESS":true}%
Mac@Flobul % curl "http://192.168.23.49/cgi-bin/sendmsg.lua?cmd=SET+SLNT+1"
{"INFO":{"CMD":"SET SLNT 1","RSP":"TIMEOUT","MSG":"Stove communication failed"},"DATA":{"NODATA":true},"SUCCESS":false}%  

Initial values : FAN OFF and POWER 1. When I send SLNT+1, POWER is set to 3. Nothing happens on SLNT+0.

Mac@Flobul % curl "http://192.168.23.49/cgi-bin/sendmsg.lua?cmd=GET+STDT"  
{"INFO":{"CMD":"GET STDT","RSP":"OK"},"DATA":{"LABEL":"WPalaControl","GWDEVICE":"wlan0","MAC":"XXXXXX","GATEWAY":"XX.XX.XX.1","DNS":["XX.XX.XX.1"],"WMAC":"XXXXXX","WMODE":"sta","WADR":"XX.XX.XX.49","WGW":"XX.XX.XX.1","WENC":"psk2","WPWR":"-65 dBm","WSSID":"OUI-FILLE","WPR":"dhcp","WMSK":"255.255.255.0","WBCST":"XX.XX.XX.255","WCH":"6","EPR":"dhcp","EGW":"0.0.0.0","EMSK":"0.0.0.0","EADR":"0.0.0.0","EMAC":"XXXXXX","ECBL":"down","EBCST":"","APLCONN":1,"ICONN":0,"CBTYPE":"miniembplug","sendmsg":"2.1.2 2018-03-28 10:19:09","plzbridge":"2.2.1 2022-10-24 11:13:21","SYSTEM":"2.5.3 2021-10-08 10:30:20 (657c8cf)","CLOUD_ENABLED":true,"SN":"000000000000000000000000000","SNCHK":0,"MBTYPE":0,"MOD":318,"VER":47,"CORE":129,"FWDATE":"2017-05-03","FLUID":0,"SPLMIN":12,"SPLMAX":51,"UICONFIG":1,"HWTYPE":6,"DSPTYPE":1,"DSPFWVER":26,"CONFIG":1,"PELLETTYPE":1,"PSENSTYPE":0,"PSENSLMAX":0,"PSENSLTSH":0,"PSENSLMIN":0,"MAINTPROBE":0,"STOVETYPE":1,"FAN2TYPE":2,"FAN2MODE":3,"BLEMBMODE":1,"BLEDSPMODE":1,"CHRONOTYPE":0,"AUTONOMYTYPE":2,"NOMINALPWR":7},"SUCCESS":true}% 
Domochip commented 8 months ago

Hi,

First, thank you, following the track, I found a bug I introduced recently (I just fixed the pre-release). Regarding your issue, The "SILENT" mode seems to be designed for Stove which have FAN3 and FAN4 (additionnal Air ducts or Water pumps).

Here is what it is doing: Set RoomFan, FAN3 and FAN4 to 0 image Your Stove has FAN2TYPE = 2 which means you have only Room Fan and then iSetRoomFan3Atech return an error.

So for your model, you just need to do a SET+RFAN+0 I will improve error messages of WPalaControl in next release, then it will return an UNSUPPORTED error.

Flobul commented 8 months ago

You're welcome.

Only ROOM FAN on my stove, exactly.

I just tested your latest fix, same cmd results than before, but now POWER is no more set to 3 anymore.