-
Notifications
You must be signed in to change notification settings - Fork 19
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
iT600 gateway rejected 'read' command with content '{'requestAttr': 'deviceid', 'id': []}' #3
Comments
I guess that my project does not support TS600 thermostats. I have added request/response debugging to the demo I am guessing the problem will be here: https://github.com/jvitkauskas/pyit600/blob/master/pyit600/gateway.py#L114 (your thermostat is probably not of a
|
I've just send you logs. I thought even if there is unsupported equipment it will be listed somewhere :) |
Can you try running |
So this is a log from new branch for ts600 after execute main.py trying to change temperature. A listed device id is the right id for ts600 thermostat. I don't understand why this key is incorrect...
|
Ok, it seems like the temperature control is different for that thermostat. I'll help you find out what's needed later. In the meanwhile, can you send me the screenshot of the mobile app showing the thermostat control screen? |
I've just run main.py several times and get two different responses. One shows error indicating probably connection error (?), and at the next shot I've got probably correct temperature change - my thermostat is configured to max temp 20oC and main.py is trying set 21oC. Despite this temperature changed from 13,5oC to 20oC (which is max). Checked it with different values and it works. The only change during last two days was that I disconnected UGE600 from power.
|
Unfortunately, the above error started reappearing. After restarting the UGE600, everything works as I wrote in the previous comment. Wonder what's going on with this UGE600 gateway... |
I recommend not using UGE600 website as it sends unencrypted requests which backend does not understand. Furthermore, I think the gateway strategy is to delay and not to send any response on incorrect data, so you might be unnecessarily DDoSing your gateway. You are correct that Did that strange error occur when you have visited UGE600 website? |
here is log after doing what you asked and run main.py with python3 sudo apt update DEBUG:pyit600:Gateway response: DEBUG:pyit600:Gateway request: POST http://192.168.1.122:80/deviceid/read DEBUG:pyit600:Gateway response: Traceback (most recent call last): kenn |
all the temperatures in the log file are correct and acording to termostat sq610RF kenn |
@dyrvigk what is the maximum and the minimum temperature you can set your thermostat to? Do you have it configurable in Salus App? |
Yes
From 5-35 degree c
tir. 15. sep. 2020 12.27 skrev Julius Vitkauskas <[email protected]>:
… @dyrvigk <https://github.com/dyrvigk> what is the maximum and the minimum
temperature you can set your thermostat to? Do you have it configurable in
Salus App?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG7RHTBKT7ILTJ4MDHRDHD3SF46RJANCNFSM4RAQ3PXQ>
.
|
@dyrvigk I have made some fallback values for min/max temperature. Get the latest files and try again. |
Hi Now it looks like something happens DEBUG:pyit600:Gateway response: DEBUG:pyit600:Gateway request: POST http://192.168.1.122:80/deviceid/read DEBUG:pyit600:Gateway response: DEBUG:pyit600:Gateway request: POST http://192.168.1.122:80/deviceid/read DEBUG:pyit600:Gateway response: Got callback for device id: 001e5e09025f4c40 DEBUG:pyit600:Gateway response: kenn |
@dyrvigk Try adding custom component from my fork to home assistant https://github.com/jvitkauskas/homeassistant_salus You might need to create |
now it works in homeassistant do yoy know if it is possibel to get battery status and maby also humidity read and how i will now try to se if the salus vs20wrf also works also salus same function exatly as the other but with 4 aaa batteries instead of rechargeable thanks a lot |
other one also working |
I would have to make some all day tests with that error. I think it occurs even if do not access UGE600 website, but... I will have to try again. Strange thing is I added printing some fields from "th" object just before place where error occurs and values were correct despite the error. As far as "set_climate_device_mode" is concerned I have mentioned that the state of thermostat action/mode is not displayed correctly in HA. I think this method could be for forcing thermostat to start heating or not. In TS600 I recognized two fields representing this state:
This is a place which could be corrected in integration with HA with this model. |
Sorry, didn't see your message. I have fixed RunningState in version 0.0.6. So you can update that in manifest.json or use newer file(s) from my fork https://github.com/jvitkauskas/homeassistant_salus |
Thanks
i will try that
kenn
<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail>
Virusfri.
www.avast.com
<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail>
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
Em qua., 16 de set. de 2020 às 22:55, Julius Vitkauskas <
[email protected]> escreveu:
… Sorry, didn't see your message. I have fixed RunningState in version
0.0.6. So you can update that in manifest.json or use newer file(s) from my
fork https://github.com/jvitkauskas/homeassistant_salus
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG7RHTFDNZJOC26AJ6MATL3SGEQ3VANCNFSM4RAQ3PXQ>
.
|
@dyrvigk does Salus app show humidity and battery status? |
hi
yes it does
kenn
Em sex., 18 de set. de 2020 às 01:17, Julius Vitkauskas <
[email protected]> escreveu:
… @dyrvigk <https://github.com/dyrvigk> does Salus app show the humidity
and battery status?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG7RHTHVWNFUHW6BEPBDZBTSGKKIFANCNFSM4RAQ3PXQ>
.
|
@dyrvigk can you provide me the output and write down the corresponding humidity and battery status? I need several samples to be able to compare the data and find out where it does report it. |
Hi sorry that i dident get back to you on this work has been holding me up i will come back to you later if that is ok kenn |
Hi, when I launch main.py with demo application and trying to connect to my UGE600 gateway I get error during "read" request. Trace log at the bottom.
The same thing when I try to integrate with home-assistant. There is a issue about that in homeassistant_salus project: konradb3/homeassistant_salus#2
Unfortunately I do not have any python experience and will be glad if someone can help figure out what is wrong.
System information:
Trace log after launching main.py:
The text was updated successfully, but these errors were encountered: