iklein99 / homebridge-smartthings

This is a plugin to Homebridge to connect your Smartthings network into Apple Home Kit.
Apache License 2.0
150 stars 52 forks source link

ACs mixing up F/C or somehow showing incorrect temps #232

Open menasheh opened 7 months ago

menasheh commented 7 months ago

Describe The Bug: ACs with farenheight seem to be coming in to homekit as 147 degrees

To Reproduce: Connect my SmartThings split units to homekit

Expected behavior: Should show the actual temperature (70), not 147

Logs: The ones that seem relevant...

[09/11/2023, 17:31:03] Homebridge v1.7.0 (HAP v0.11.1) (Homebridge) is running on port 51281.
[09/11/2023, 17:31:04] [SmartThings] Restoring existing accessory from cache: Menasheh AC
[09/11/2023, 17:31:04] [SmartThings] Restoring existing accessory from cache: Living Room AC
[09/11/2023, 17:31:07] [homebridge-smartthings-ik] This plugin generated a warning from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 47 exceeded maximum of 38. See https://homebridge.io/w/JtMGR for more info.

Plugin Config: access token and 0 for poll sensors. No extra sensor, no humidity sensor. 2 irrelevant devices ignored.

{
    "bridge": {
        "name": "Homebridge BE86",
        "username": "...",
        "port": 51281,
        "pin": "...",
        "advertiser": "avahi"
    },
    "accessories": [],
    "platforms": [
        {
            "name": "Config",
            "port": 8581,
            "auth": "form",
            "theme": "auto",
            "tempUnits": "f",
            "lang": "auto",
            "platform": "config"
        },
        {
            "token": "...",
            "platform": "smartthings-tv"
        },
        {
            "name": "SmartThings",
            "BaseURL": "https://api.smartthings.com/v1/",
            "AccessToken": "...",
            "GarageDoorMaxPoll": 0,
            "PollLocksSeconds": 0,
            "PollDoorsSeconds": 0,
            "PollSensorsSeconds": 0,
            "PollSwitchesAndLightsSeconds": 0,
            "PollWindowShadesSeconds": 0,
            "ExposeHumiditySensorForAirConditioners": false,
            "OptionalModeForAirConditioners": "Speed",
            "IgnoreDevices": [
                "Third AC",
                "Samsung Q50 Series (32)"
            ],
            "platform": "HomeBridgeSmartThings"
        }
    ]
}

Screenshots: 95272 633

Environment: