search

mathieu-mp / homeassistant-intex-spa

Home Assistant integration for Intex Spa
https://github.com/mathieu-mp/homeassistant-intex-spa
MIT License
32 stars 6 forks source link

Current temperature sensor #36

Closed Elkropac closed 1 year ago

Elkropac commented 1 year ago

Hi, do you think there should be "current temperature" sensor, so you can create graphs easily? Maybe as a disabled entityby default?

I wanted to make graph and ended using template sensor

template:
  - sensor:
      - name: "spa_temperature"
        state: "{{ state_attr('climate.spa', 'current_temperature') | round(1) }}"
        unit_of_measurement: "°C"

Works for me.

If you find separate sensor useful, i can try to make it as PR

mathieu-mp commented 1 year ago

Hi @Elkropac, still good to read your contributions !

Both ways are good: template or dedicated sensor. I'll add a Temperature Sensor, disabled by default: #37

Elkropac commented 1 year ago

Hi, thanks you have already implemented it . I was hoping to do it myself to refresh the knowledge how to do it ;) I'm not complaing . Thank you very much!