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

Please elaborate on why current temperature is disabled by default #50

Closed verwilst closed 1 year ago

verwilst commented 1 year ago

Currently the readme says the current temp. sensor is disabled by default. Would it be possible to update that part of the documentation why exactly that is, and maybe what the 'risks' are when enabling it?

mathieu-mp commented 1 year ago

Hi @verwilst,

Thank you for your interest and relevant question.

The current temperature info is already available as an attribute of the climate entity, and can be exposed as a status via a template. There is no risk or side effect to enable the current temperature sensor feature. It was added following #36, to ease monitoring or graphing. It has not been enabled by default as it duplicates already existing data in the home assistant database, while most of the users won't need it.

I will update the docs with this mention.

Regards, Mathieu.

mathieu-mp commented 1 year ago

Implemented by #60