-
Notifications
You must be signed in to change notification settings - Fork 71
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
[BUG] Thermostat mode not changeable on the app #695
Comments
Thanks for registering the issue, can you maybe get some more information about that device type? If you log in to the domoticz site, then go to this url (change first part): Plz then find the part of that thermostat, it should look like this:
|
Ok here is that info:
|
Mm thanks that's handy information thanks. I am wondering what kind of api call is being done when you change the mode in the web ui. |
Ok I checked and for example to turn that thermostat to "cool" it would be:
|
ok thats funny as it uses the name instead of the 0/1/2/3 id. |
Great thanks a bunch.... let me know how I can help or test for you |
mm i expected type to be Thermostat to be fair.. thats kinda weird. and my other question would be, are these modes always 0 based auto numbering? why wouldnt it then be I dont really get why they didn't implement the same level logic as they have for for example selectors |
Do you know if these command also work to set the mode? login to web domoticz, then try these commands to see if they work: https://127.0.0.1/json.htm?type=setused&idx=119&tmode=2 just play around with those tmode values 🗡️ |
https://127.0.0.1/json.htm?type=setused&idx=119&tmode=x always just returns Also weird I can return the mode number with It errors with: |
ok did some ground work, aldo the command url is not ready yet. can you maybe find out what properties are really needed to work (like description.. thats also weird) |
i'm uploading some new version here on a alpha track, please enlist as tester so you get the test version if available: (Choose the app that you have, i guess the second link) |
After testing the url: I found out that it works without the I also signed up for the beta on the domoticz premium app. |
FYI In current domoticz beta's |
Thanks @kiddigital I tried that format and it fails. I read #692 and the forum post and see there is only backward compat on the next couple of future versions. |
@kiddigital is setused also backwards compatible? |
ok so if i understand you correctly, these calls should work: so used = true is also mandatory |
Yes, see domoticz PR #5720 |
Correct, https://127.0.0.1/json.htm?type=setused&idx=119&tmode=0&used=true works. FYI also reading through the forum posts the backward compat will only be there for 12-18 months or so. |
@user8446 looks like the alpha update should be in the store, i think it should at least show the mode in a dropdown combobox on the utilities page. could you check and make screenshot? Mmm i uploaded it to closed alpha as its totally untested code.. i need your google email to whitelist you for alpha testing 🍡 |
Ok just sent a msg through the forum |
Ok just updating you I just got an update to 0.2.359 (9332) but haven't received 0.2.360 (9336) yet |
Ok I received version 0.2.360 (9336). Here is the screenshot: After selecting a mode, I click on "set" and this pops up and then "ok" to try to set it. Only the mode "off" works. The other modes do not change but I see a log entry that something was sent to that thermostat:
|
ive send a new version to google on the closed test track |
plz check the latest version |
Ok this morning I just received 0.2.363 (9371) from 0.2.360 (9336) - I never received the 0.2.362 (9355). The UI and behavior is the same. All selected modes will send an "off" through Domoticz and the UI will then display "cool". |
Is it maybe possible to give me access on your system to that device? so i could try it out? |
No problem sending a msg through the forum with the credentials now |
thanks i did some more changes, new version is on tha way |
yes i indeed fixed the UI, also for the dashboard (if people put it on dashboard.) i actually tested that, copied the used http url and tested if that worked via the browser, as i could not really login to your system with the mobile app. dont you see any log about the mode in domoticz?? |
It changes in the app but nothing in the logs on this latest version. On the previous version it would show the command in the logs even though it would only turn to mode "off" |
This is not a bug in the mobile app, it is -a very old- bug in Domoticz/openzwave. The mode is sometimes interpreted as an index into an array of available modes and other times as the code for the mode. Off happens to be both code 0 and the first entry in the table, Heat has code 1 and is the 2nd entry, but for many other modes there is no match between the code.and the position in de he array. In openzwave/Domoticz is a check that silently ignores any invalid codes: thus no logging is shown when the bug is triggered. The same behavior of ignoring setting some modes can be observed in Domoticz -without the mobile app involved-. |
Thanks for the info but that wouldn't be the issue here. Changing the modes with the Domoticz web front end works without fail. The API call is exactly the same as what the web front end uses. It doesn't matter whether it's from a script, app, or the web front end itself. Today I received v0.2.362(9351) - not sure why the version # went backwards but no change in behavior. |
i assume still the case. dont really get why the api doesnt work, but its not really possible for me to test this |
Unfortunately still not working. API still works though. |
Hi just reporting that my Z-Wave thermostat modes are changeable on the Domoticz web UI but on the Domoticz mobile app. The mobile app reports the correct mode that it is in but does not have an option to change it. Any direction to debug, test, or solve?
Thermostat mode changeable on the web UI:
Thermostat mode not changeable on the mobile app:
Smartphone (please complete the following information):
The text was updated successfully, but these errors were encountered: