teodoradima / homebridge-samsung-air-conditioner-smart-things

Apache License 2.0
28 stars 16 forks source link

Unable to add ACs to homebridge #6

Open lithos2003 opened 3 years ago

lithos2003 commented 3 years ago

When trying to use this plugin to add my Samsung ACs to homebridge, it does not poll/find the ACs at all. Maybe I'm missing a prerequisite on my system to make this work?

Here are the logs:

10/26/2021, 8:24:24 AMSamsung Air Conditioner SmartThings Bridge starting 10/26/2021, 8:24:29 AMSamsung Air Conditioner SmartThings BridgeLoaded plugin 'homebridge-samsung-air-conditioner-smart-things' 10/26/2021, 8:24:29 AMSamsung Air Conditioner SmartThings BridgeLoading 1 platforms... 10/26/2021, 8:24:29 AMSamsung Air Conditioner SmartThings Bridgehomebridge-samsung-air-conditioner-smart-thingsFinished initializing platform: 10/26/2021, 8:24:29 AMSamsung Air Conditioner SmartThings BridgeBridge is running on port 51836.

and here is the plugin config:

{ "platform": "HomebridgeSamsungACSmartThings", "name": "homebridge-samsung-air-conditioner-smart-things", "token": "tokenz", "temperatureUnit": "F" }

I'm running this on a Hoobs box running the following versions: Hoobs 4.1.11 Node 14.18.1 Homebridge 1.3.5

manolab commented 2 years ago

I'm in the same situation: did you find a solution? Seems as the plugin isn't executed at all... Just this in the logs:

[27/05/2022, 07:20:16] [homebridge-samsung-air-conditioner-smart-things] Initializing HomebridgeSamsungACSmartThings platform... [27/05/2022, 07:20:16] [homebridge-samsung-air-conditioner-smart-things] Finished initializing platform:

Any help? Homebridge 1.4.1 Node 14.18.3

manolab commented 2 years ago

I've studied the code, the plugin adds only Acs with a specific name. Don't know why at all. That name is: [room a/c] Samsung.

First lines of https://github.com/teodoradima/homebridge-samsung-air-conditioner-smart-things/blob/4c6405a9e0c31c89bf0b7dec0ebcab46c082e143/src/samsungApi.ts

min0628 commented 2 years ago

Hi, I'm in the same situation. @manolab did you solve this problem?

manolab commented 2 years ago

I tried to modify the code based on the response the SmartThings API gave with my ACs: it worked showing the 3 ACs in my house but crashed shortly after when trying to display/modify the parameters of the ACs... It's not a universal plugin at all. Much more development seems to be necessary.

LPFchan commented 2 years ago

Even tried changing AC's name to [room a/c] Samsung but I've had no luck :(

StoreEverything commented 2 years ago

const AC_DEVICE_NAME = '[room a/c] Samsung';

doesn't it need to be

const AC_DEVICE_NAME = 'Samsung Room A/C';