adrianjagielak / homebridge-futurehome

The best and only Homebridge plugin to integrate Futurehome devices into HomeKit.
Apache License 2.0
1 stars 0 forks source link

Thermostats and termometers not grouped together properly #1

Open sfla opened 1 day ago

sfla commented 1 day ago

These are the controls from a single Futurehome thermostat. I have five of these, so I got 25 controls.

image

The expected outcome would be a single square control, showing the current thermometer-temperature instead of 0,0°, and what it's set to.

In this picture, the top left thermometer is correct (17.5 is room temperature now), and the top right thermostat is correct (in that the thermostat is set to 19.0) but wrong in showing 0,0.

The bottom three are unused channels (ch0, ch3, ch4) that I'm fine with just hiding myself (not sure where it gets 21 from though). But I want a way to merge the thermometer and thermostat into an actual thermostat-control.

Like this

image

I recognize that different thermostats may be set up differently, and by using different channels for different things. But a way for me to manually enter which channel should be used together like this would be perfect. If that's how it works. I have no idea.

adrianjagielak commented 1 day ago

Hi,

First off, wow, I almost forgot about this plugin after almost a year of inactivity lol

Could you please enable Homebridge Debug Mode in your Homebridge settings, restart the plugin, and then download and share the logs with me? I'm particularly interested in two lines within the logs: one that starts with "Got devices info: (...)" and another that starts with "Got devices state update: (...)".

I'm curious about the structure of the info/state so each thermostat appears as five separate accessories in HomeKit, but it correctly appears as a single accessory per thermostat in Futurehome.

StianFlatby commented 1 day ago

Thanks! devicesInfo.txt devicesStateUpdate.txt

I do see that I have five ch_0 units in the Futurehome-app that aren't assigned to any rooms (two of them weirdly named for some reason), so Futurehome isn't perfectly mapped either. Looks like I'm only using ch_1 and ch_2 for all rooms, in both apps. I have no idea what they correspond to, if it's the type of sensor, like "room vs floor" or whatever..

Regardless, I don't mind the extra accessories, I can hide them. But I do really want to make the two actual accessories, thermostat (ch_1) and thermometer (ch_2), act as a single working thermostat if possible.

adrianjagielak commented 18 hours ago

Okay, I looked into it.

It’s odd that your thermostats appear as five separate devices each in the API. However, since it’s a Z-Wave device, I don’t have as much experience with them as I do with Zigbee devices (even when I worked at Futurehome, I mostly worked with Zigbee, lol).

But I’ve made some changes so that now (at least for a "zw_411_3_515" model thermostat), the five separate devices are merged into a single device, and each one has its own Thermostat HomeKit service.

I also did a small refactor of the thermostat service itself in my plugin. Now, it only shows the supported modes for the given thermostat (e.g., heat and off instead of heat, cool, auto, and off).

I’ve pushed these changes in the 0.3.0 update of the plugin. You need to delete and reinstall the plugin, as there might be caching bugs due to device-merging changes when updating directly from 0.2.0. Let me know if it works!

adrianjagielak commented 7 hours ago

Here's how it looks like when I hard-coded your device info/state into my local plugin instance:

IMG_5884

IMG_5885