-
Notifications
You must be signed in to change notification settings - Fork 413
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
Request to support another thermostat please #305
Comments
I think to fully support this, it will also need some scripting in the UI, to selectively display one of 3 temperature controls depending on whether the thermostat is in auto mode, and whether temporary hold is set. |
Thank you for the support, I have updated the integration, when I'm adding the thermostat it shows that garage opener |
Can you send the log message with the dps list. There may be some missing compared with the list above. |
Original config copied the value rather than the id from issue #305
Nevermind, I found a transcription error in the config. |
I have updated the integration from main branch, still it shows as garage opener here is the log
|
It seems dp 27 is missing, so the temperature correction needs to be removed from the config. |
I have done as you said, I have commented the dp 27 and it is functioning as thermostat, I will confirm the rest once I return back to home. Thanks again :) |
I have tested it, I can't set the temperature it needs to be DPS 104 for C and DPS 106 for F Also I get warning in the logs 2022-12-14 21:04:44.006 WARNING (MainThread) [custom_components.tuya_local.generic.climate] Unrecognised HVAC Mode of cold ignored as follows for ETOP thermostat
Thanks |
The log message was coming from the Auto thermostat when the mode was not auto. It should have mapped cold to off for that thermostat. The main thermostat has the temperature mapped to dp 104(c)/106(f), so should be working. |
I have checked again with the new update still there is log
Also I have discovered that if the thermostat is on cold mode it uses dps 104 for setting temp in C and 106 for setting temp in F, however in Hot mode it uses dps 105 to set temp in C and 107 to set temp in F I'm not able to change the temp using dps 16 or dps 17 by the way, can you support a normal socket device? |
In heat mode, different dps are used to control temperature than in cool mode. Rather than create a fourth climate entity for this case, merging the Auto and main climate entities allows all cases to be covered (albiet with redundant second temperature setting in cool and heat modes). May also fix the error message "Unrecognised HVAC Mode of hot ignored", which I think may have been coming from the Auto thermostat not applying the default mapping. Issue #305
I have merged the Auto and main thermostats back into a single thermostat now to handle the different heat and cool dps. |
for setting the temperature it will not register fast, sometimes it will revert to the previous set temp |
Maybe you can try on 2023.1.1, where the device state feedback is much quicker. But this may be a limitation of the dual temperature thermostat card in HA. |
I have tried, the fan and presets registers very fast, however the temperature still lags |
OK. I think I will close this, as the fan and presets show that the communication with the device is working fine. It is probably a limitation of that dual temperature UI component that it is slow to send commands due to waiting to see if you adjust the other temperature, or something. |
As for "sometimes it will revert to the previous set temp", while I was making the last changes, I did come across some code that was holding onto the sent values for up to 20 seconds, and there was a possibility that they would be resent by a future send command. That should be fixed now. |
Hello, Thanks again for supporting these devices, I came with new Thermostat.
While trying to add the thermostat it shows as switch only.
The thermostat is from a company called etop controls here is a link of the product
The below is what I can read from the device:
As for the DPS these what I have found out to be (I'm using tinytuya in a jupyter notebook and testing it):
The text was updated successfully, but these errors were encountered: