search

jasondefuria / Honeywell-Lyric

Honeywell Lyric for Home Assistant
9 stars 3 forks source link

Include remote sensors in devices #2

Open tasmin opened 4 years ago

tasmin commented 4 years ago

I have the T9 thermostat with a remote sensor in my bedroom. The only way I have been able to see the data from the remote sensor is via the API on the Honeywell developer website, by using the API to get rooms:

https://api.honeywell.com/v2/devices/thermostats/{deviceID}/group/{groupID}/rooms

The response shows both the main thermostat status and the remote sensor status (they show up as accessories in the rooms).

Can we change the list of devices to be the accessories listed in the above mentioned API Response and use them in Home Assistant?

I am an experienced developer, but I am not a python expert. I can help if you need, but I will need some guidance.

Thanks, Tasmin.

jasondefuria commented 4 years ago

@Tasmin. Interesting- didn't know the new ones have this sensor. What does the sensor do? Occupancy?

Will look into this, though can't promise how soon. I, myself am new to python and hopefully going to work on #1 first. It seems like #1 will require a rewrite using API as well. Once that is working, then it will be time to implement this.

tasmin commented 4 years ago

The remote sensor has humidity, temperature, motion, occupancy, battery status and some settings.

Let me know if I can help. I looked at the code but got lost in all the imports and method calls. I don't see where the API is called, is that in the lyric module in the core?

Here is the response to the rooms API:

{ "deviceId": "MY-DEVICE-ID", "groupId": 0, "rooms": [ { "id": 0, "name": "Thermostat", "type": "Other", "avgTemperature": 74.414, "avgHumidity": 55, "accessories": [ { "accessoryId": 0, "accessoryAttribute": { "type": "Thermostat", "connectionMethod": "WI", "name": "Thermostat", "model": "RCHT9610WFW2004", "serialNumber": "MY_SERIAL_NUMBER", "softwareRevision": "1.2.8.0", "hardwareRevision": "FLYCATCH_TS-0001" }, "accessoryValue": { "coolSetpoint": 0, "heatSetpoint": 0, "indoorHumidity": 57, "indoorTemperature": 74.664, "motionDet": false, "occupancyDet": false, "excludeTemp": false, "excludeMotion": false, "pressure": 0, "occupancyTimeout": 0, "status": "Ok", "batteryStatus": "Ok", "rssiAverage": -41 } } ] }, { "id": 1, "name": "Master Bedroom", "type": "Bedroom", "avgTemperature": 74.336, "avgHumidity": 51, "accessories": [ { "accessoryId": 1, "accessoryAttribute": { "type": "IndoorAirSensor", "connectionMethod": "RL", "name": "Master Bedroom", "model": "4352", "serialNumber": "", "softwareRevision": "1.0.3.0", "hardwareRevision": "48.49.45.65" }, "accessoryValue": { "coolSetpoint": 0, "heatSetpoint": 0, "indoorHumidity": 51, "indoorTemperature": 74.609, "motionDet": true, "occupancyDet": false, "excludeTemp": false, "excludeMotion": false, "pressure": 0, "occupancySensitivity": "Medium", "occupancyTimeout": 0, "status": "Ok", "batteryStatus": "Ok", "rssiAverage": -45 } } ] } ] }

Thanks, Tasmin.

On Wed, Aug 5, 2020 at 12:53 PM jasondefuria notifications@github.com wrote:

@tasmin https://github.com/tasmin. Interesting- didn't know the new ones have this sensor. What does the sensor do? Occupancy?

Will look into this, though can't promise how soon. I, myself am new to python and hopefully going to work on #1 https://github.com/jasondefuria/Honeywell-Lyric/issues/1 first. It seems like #1 https://github.com/jasondefuria/Honeywell-Lyric/issues/1 will require a rewrite using API as well. Once that is working, then it will be time to implement this.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jasondefuria/Honeywell-Lyric/issues/2#issuecomment-669461312, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMBCLCFQQKVN5HDILONZ53R7G2B5ANCNFSM4PU7BMYQ .

jasondefuria commented 3 years ago

@tasmin @Timmo001 has made a pending PR for native integration. Try to install and see if it polls that sensor. This will become deprecated once the native integration occurs. https://github.com/home-assistant/core/pull/39695

tasmin commented 3 years ago

@tasmin @timmo001 has made a pending PR for native integration. Try to install and see if it polls that sensor. This will become deprecated once the native integration occurs. home-assistant/core#39695

I finally got the branch code from timmo001's repo branch "honeywell-lyric-two" working in my custom_components/lyric folder. But I still do not see the remote sensor listed in the list of devices.