lukas-clarke / eight_sleep

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

Enhancement - Add current temp settings for each side #14

Closed rsteckler closed 6 months ago

rsteckler commented 6 months ago

Love that you re-created this and got it working again.

Can you add the current temp setting for each side?

lukas-clarke commented 6 months ago

To clarify, you want to see the bed cover temperatures as sensors? There are temperature sensors already, that were inherited from the previous integration. Are those not reliable/accurate for you? What unit would you like to see the temperature in? What unit are you seeing the current bed temperature sensors in?

rsteckler commented 6 months ago

I'd like to see the target setting. This is switchable in the Eight app to either an arbitrary -10 to +10, or as a target temperature.

Right now, the service calls to increment or set a target are write-only. There's no way to see what the current setting is.

Here's a screen from the app:

image

I believe the "-3" represents the current temperature that you currently expose. I'm looking for the target of "10" as a new sensor.

lukas-clarke commented 6 months ago

Oh, understood. I would like to update the integration to be able to natively edit the device instead of using service calls. But this would require a larger re-write and I'm pretty busy for the near future

rsteckler commented 6 months ago

First, I totally get it. Working on an open source project while also having a "real" job, family, hobbies etc is tough. Been there.

Second, nevermind. I just realized the setting is an attribute of the bed state. I was just expecting it to be an entity. My bad!

Closing :)