dolezsa / thermal_comfort

Thermal Comfort sensor for HA (absolute humidity, heat index, dew point, thermal perception)
Other
619 stars 118 forks source link

Filter by desired types instead of undesired sensors #396

Open igorsantos07 opened 2 months ago

igorsantos07 commented 2 months ago

The sibling Comfortable Environment Card has a more direct approach when asking for the temperature and humidity sensors: it simply filters by those attributes, instead of rulling out anything that "might not be useful" (which could then cause more issues in the future, such as #393).

Is there any reason for going the long way instead of the simpler one? This still causes my setup to show a lot of irrelevant sensors on the config flow. Would there be scenarios where the temperature sensor would not be identified as such, for instance?

dagobert commented 1 month ago

@dolezsa Is there a reason why you do a blacklist instead of a whitelist?