-
Notifications
You must be signed in to change notification settings - Fork 18
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 Not Working #203
Comments
Message that will be displayed on users' first issue |
@carTloyal123 Can you take a look at this? |
Sure thing, you can assign it to me |
@Barba4life it looks like things are actually working as they intend but might need to be adjusted. Could you describe more for me exactly what you are doing? It looks like you are setting your temperature to 80F as the target temperature. Is that correct? This is actually being clamped by the plugin because the underlying homebridge platform only allows a max value of 25C for cooling setpoints. I would try setting the same value to something like 70, then 75 and see if that behaves correctly. Do that for me and in the mean time I will look into this upper limit being an issue. |
It looks like we can customize this a bit actually for our Wyze thermostat. @Barba4life could you actually just do me one favor and tell me the min and max values you can set the thermostat to when it is in heating mode and when it is in cooling mode and when it is in auto mode? This should be 8 numbers total: 2 for heating only, 2 for cooling only, 4 for auto mode (2 for the heat setpoint and 2 for the cool setpoint). With these values we can adjust the limits in this plugin. Thanks and let me know if you need more detail! |
Absolutely, I should have some time today. Thank you @carTloyal123 |
Sorry didn’t mean to close it, fat fingers over here on my cell phone… 😅 |
@carTloyal123 I am moving as much as I can out of WyzeAccesory.js. If you have any questions please let me know. |
Oh I see that now. Um I will work on moving stuff over while I am waiting to hear back on these tuning values. Is there a good reference file you have updated that I can follow? |
Looks like everything has been updated except the thermostat so I spoke a little too soon and there are plenty of examples. I will work on moving things into the thermostat file |
I added the refactor to main branch in my fork: https://github.com/carTloyal123/homebridge-wyze-smart-home/tree/main I can not test the changes since I am not currently using my wyze thermostat at home so @Barba4life maybe you can install a beta version or share your device on the wyze app with me for testing. Let me know! |
Thank you @carTloyal123, when beta is ready I’ll test. I checked the thermostat, the lowest setting is 35F and highest is 105F degrees. I also only use the heat option on the thermostat sorry. (I could also share you my device on Wyze if you wanted to see all the options too) |
@carTloyal123 I was able to add the Wyze plugin to HB. Created a child bridge and connected it to Homekit. Very smooth. However, I don't see the thermostat in Accessories in HB. Here's the log: [2/10/2024, 4:20:57 PM] [Wyze] Launched child bridge with PID 32559 Any insight you can give me would be very helpful. Plugin Version: v0.5.42 I'm brand new to Github. Apologies in advance if I'm not asking the right question or providing enough context. |
You might want to try the latest release. |
Even better it looks like I can not login in through the plugin so I am troubleshooting that. I suspect it is something with my specific Wyze account but we will see. Once I get through that I will see what I can find |
@pughpypants go ahead and open a new issue in Github and site your findings there. Your issue seems unrelated to this thread. Welcome to github, glad to have you and thank you for the feedback! |
Thanks @carTloyal123. Will do. |
Ohh no I did the update, now my child bridge keeps restating. Hey @carTloyal123 what email should I share the thermostat to through the Wyze UI? So far I tried uninstalling and reinstalling, still same problem. |
You can share it with Looks like this is a pretty serious issue. I will look into it hopefully today or tomorrow, at worst this weekend. Thanks for the extra input here! |
The logs above are not even from this project they are from zswuwing This is the project they are from. |
The very first set of logs. If you look at the error it reports the node it is using. Which is a branch of this one from zswuwing |
That is pretty weird but makes more sense. The other issue linked seems to be using the correct version and still has the same issue so I'll give it a look but that is a great catch. @Barba4life definitely see about getting the correct version and then report back for us if issues persist |
New release out. |
@Barba4life what is the status of this with the latest release of the plugin for you? |
Sorry for the late update. That did the trick, it no longer crashes. as for the setting correct temp, I don't see a difference. Still able to at least turn on the heat mode, and turn off. |
Is there still more work for this? @carTloyal123 @Barba4life |
I am good here. Might need to merge in my updates for the thermostat from my fork but sounds like @Barba4life is up and running so nothing more for now. We can close! |
Describe The Bug:
In the home app device shows up perfectly. Currently only able to set thermostat to "off", everything else does not work.
Please help, thank you for reading.
To Reproduce:
In the home app set temperature, app will say setting temperature.
Expected behavior:
Never sets the physical thermostat...
Logs:
Plugin Config:
Screenshots:
Environment:
Synology NAS
The text was updated successfully, but these errors were encountered: