lukas-clarke / eight_sleep

Home Assistant Eight Sleep integration that works with Eight Sleep's new API and OAUTH2
MIT License
89 stars 17 forks source link

Use it as a climate entity #38

Open v3rm0n opened 1 month ago

v3rm0n commented 1 month ago

Would it be possible to configure a side as a climate entity? Then it would be possible to define a nice looking thermostat for it in the UI

lukas-clarke commented 1 month ago

I'm not sure a climate entity is the best match for this. As you can see from the docs (https://developers.home-assistant.io/docs/core/entity/climate/) there is only one attribute from the climate entity that applies to the 8 pod. I think a better approach would be to create a device for the eight sleep, which is on the TODO section of this repo. I'll try and bump this up on my personal TODO list, now that I know there is demand for it.

v3rm0n commented 3 weeks ago

Thanks!

With the end goal of having a thermostat, I'm just wondering if creating a device would help here?

I don't know much about HA internals, but to define a generic thermostat you need a sensor and a switch so that HA would toggle the switch according to the sensor value, which would not work with Eight Sleep.

The other way to define a thermostat is to implement a climate entity.

I'm thinking that it should probably work similarly to a wifi thermostat where you set the temperature and the climate device will take care of actually achieving it while also reporting the actual temperature, like this https://github.com/home-assistant/core/blob/dev/homeassistant/components/radiotherm/climate.py