search

lynxcs / homebridge-ct200

Homebridge plugin for Bosch Easycontrol CT200
MIT License
21 stars 2 forks source link

Low Temperature (Below 10C) #23

Closed Barclay2k closed 2 years ago

Barclay2k commented 2 years ago

Describe The Bug: If the thermostat is set to temps lower than 10C, it errors in Homebridge.

To Reproduce: EasyConnect schedule has my thermostat default to 5C when I leave the house, or when it's off, and anything lower than 10C errors in HomeBridge.

Expected behavior:

Logs: [06/01/2022, 21:29:50] [homebridge-ct200] This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 5 exceeded minimum of 10. See https://git.io/JtMGR for more info. [06/01/2022, 21:29:50] [homebridge-ct200] This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 5 exceeded minimum of 10. See https://git.io/JtMGR for more info. [06/01/2022, 21:29:59] [homebridge-ct200] This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 5 exceeded minimum of 10. See https://git.io/JtMGR for more info. [06/01/2022, 21:30:00] [homebridge-ct200] This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 5 exceeded minimum of 10. See https://git.io/JtMGR for more info. [06/01/2022, 21:30:08] [homebridge-ct200] This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 5 exceeded minimum of 10. See https://git.io/JtMGR for more info.

Plugin Config: n/a


n/a

**Screenshots:**
n/a

**Environment:**

* **Plugin Version**:v2.2.0
* **Homebridge Version**: <!-- homebridge -V1.3.9 -->
* **Node.js Version**: <!-- node -v16.3.1 -->
* **NPM Version**: <!-- npm -v8.3.0 -->
* **Operating System**: <!-- hb-service -->

<!-- Click the "Preview" tab before you submit to ensure the formatting is correct. -->
lynxcs commented 2 years ago

Homebridge by default sets the temperature range to 10C - 100C, until now I didn't see a need to modify that. But since at least some people use temperatures less than 10C I'll change the range to 0C - 100C. 😄

Barclay2k commented 2 years ago

Thanks :-)

lynxcs commented 2 years ago

After a few patches I believe it's working fine now 😄. (V2.2.3) Ended up changing only the lower range, so you should not have issues when the temperature is set to 5C.

Closing. Please comment and re-open if the issue re-appears.

Barclay2k commented 2 years ago

Thanks. That works well, and errors cleared.

But I thought I would experiment in the other direction. The EasyConnect App/CT200 only allows temperature settings between 5C and 30C. Currently the homebridge plug-in let’s you go higher, so if you set the temp via HomeKit to 30.5C or higher… it’s not an accepted value and crashes the plug-in in homebridge. I presume the 5C to 30C is the same for all, are you able to put a 30C limit?

Simon

Sent from my iPhone

On 7 Jan 2022, at 21:25, Domas K. @.***> wrote:

 Closed #23.

— Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.