tetele / hvac_group

Group together `climate` components for easy control
MIT License
23 stars 5 forks source link

Sensors (domain sensor) as temperature.sensor #31

Closed gd360 closed 8 months ago

gd360 commented 8 months ago

Checklist

Is your feature request related to a problem? Please describe.

Hello,

I just tried adding my first hvac_group. It seems that you can add an actual pure temperature sensor as the sensor of a group. Is this wanted behaviour? I could add a thermostat (domain climate) but I have an sonoff sensor that runs under the domain "sensor".

kind regards Gerhard

Describe the solution you'd like

You can add a temperature sensor from domain sensor

Describe alternatives you've considered

cant think of any

Additional context

no context needed imo

tetele commented 8 months ago

Hey! Some thermostats are generic and already use such a "pure" temperature sensor. I don't see any issue with using such a sensor for a HVAC group, even more so since you're not limited to using them (you can always use a climate entity temperature sensor).

You can use either one of them. The only difference it will make is that the resulting group will:

If you don't have any common heater/cooler and if you didn't check the "toggle <heaters/coolers> on or off depending on ammbiental temperature", then the result should be the same.

gd360 commented 8 months ago

Hallo,

thanks a lot for the fast help. I am still pretty new to this. Just to be sure, I wasnt able to add a generic sensor to the helper, as it limited to "climate" entity. But now I saw that there is a fork for an external temp sensor, so is this perhaps exactly what I wanted then?

tetele commented 8 months ago

Appologies, I forgot that I haven't merged that feature :) Since you're the first person to use the helper that I interact with, I have been on my own so far.

So, to answer your initial question, now that I understand it better: yes, there is support for temp sensors and I've just merged them.

tetele commented 8 months ago

Fixed in #33

gd360 commented 8 months ago

thank you!

MasterDevwi commented 6 months ago

Hi @tetele! Which device classes are supported? I've created a template sensor with the "temperature" device_class, but the UI isn't listing the sensor as one of the available options. Should I be specifying a different device_class?

tetele commented 6 months ago

@MasterDevwi it works for me with a template sensor with device_class: temperature, state_class: measurement and unit_of_measurement: °C. Could you paste the full list of attributes of your sensor (from dev tools)?

MasterDevwi commented 6 months ago

@tetele Thanks! Apparently I was missing the state_class. Once I added it it showed up.