-
Notifications
You must be signed in to change notification settings - Fork 7
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
This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value. #32
Comments
Looks like you have a target heating temperature set higher than 30C (86F). Apple/HomeKit allows you to specify a target temperature between 10-38C (50-100F), but the native Daikin app only allows target heating temps between 10-30C. I coded the plugin to only allow the same 10-30C range to avoid the possibility of breaking the Daikin unit. If you switch the mode to Heat and then set the target temp to 30C or 86F (or lower) then the warning should go away. When cooling there's a similar range of 12-32C (53-89F) that you're restricted to setting the target cooling temp to. |
This was my suspicion, but... My thermostat is in Auto mode. My daytime schedule has a range of 64F-87F. Although the 87F clearly exceeds the 30C, this is the temperature for cooling not heating. (The Daikin range max on my thermostat and Daikin app is 90F, or 32C.) The messages start when the schedule (automatically) changes to daytime. The Daikin One is changing it, not Homekit/Homebridge. When the schedule changes to evening, the error is no longer appearing, because the range max falls below 30C. Would this be a plug-in, Homebridge, or Homekit issue? |
Ahh, interesting. I'm not sure where the issue would lie at this point. I'll need to recreate it here and see exactly where it is generating the warning. I bet I'll find a way to work around it. |
I see the same warning on startup. I also can’t change temperatures, though reading data from the thermostats and sensors works fine. I’ve tried resetting the HB cache and that was no help either. I’m also using auto mode, with both cooling and heating thresholds set at once. |
Sorry for the slow response. I was able to rework the thresholds and auto mode so that it'll actually set the two thresholds as expected in the next release. I discovered a regression that caused it to stop working. |
No worries at all! The prior version is working properly less the enhancements added in 3.1.2, and I'm sure we all appreciate you building this in the first place. Thanks for looking into it (and the other related issue!) |
@jonnyborbs Just published 3.2.0 which should fix the thresholds. @garyhooper let me know if you're still seeing issues with this version. (I didn't have much time over the past year for the plugin so I'm fuzzy on what might still remain to be fixed, if anything.) |
@jeffschubert, I've upgraded to 3.2.0, and I'm still seeing the problem:
|
@garyhooper I think it clicked what's going on. A couple other issues people have had with setting temps while in Auto mode coupled with your issue made me realize that HomeKit may always be using the heating threshold as the target temp. But my plugin right now returns the cooling threshold when the target temp is requested in Auto mode. I'm pretty sure if I have it return the heating threshold instead that'll fix it for you. I'll make that switch and we can see if that fixes it. |
Heyo, this error started popping here just recently (possibly after the most recent Homebridge UI update?) [15/09/2024, 13:45:55] [homebridge-daikin-oneplus] This plugin generated a warning from the characteristic 'Current Temperature': characteristic was supplied illegal value: number 255 exceeded maximum of 100. See https://homebridge.io/w/JtMGR for more info. Same error, but "Current Temperature" is the one throwing it |
Describe Your Problem:
In the Homebridge status window, I am seeing this message, repeating:
This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 31 exceeded maximum of 30.
Any ideas?
Environment:
The text was updated successfully, but these errors were encountered: