search

swingerman / ha-dual-smart-thermostat

The `dual_smart_thermostat` is an enhaced verion of generic thermostat implemented in Home Assistant. It uses several sensors and dedicated switches connected to a heater and air conditioning under the hood.
https://github.com/swingerman/ha-dual-thermostat
Apache License 2.0
121 stars 20 forks source link

[chore] warning doesn't say which sensor is stalled #286

Closed swingerman closed 1 week ago

swingerman commented 1 week ago

I had a Quick Look through the commit and I couldn't see if the logging will include information about which entity is causing the issue in the log, i.e:

01.194 WARNING (MainThread) [custom_components.dual_smart_thermostat.climate] Sensor is stalled, call the emergency stop

This doesn't say which sensor is stalled, and which entity the emergency stop has been called on.

I realise hvac_action_reason is set, but often this is empty by the time I want to look at it, which is why it would be good to log it so I can correlate it with an issue.

Originally posted by @jacobw in https://github.com/swingerman/ha-dual-smart-thermostat/issues/268#issuecomment-2328488207

swingerman commented 1 week ago

This one will make it more informative

jacobw commented 1 week ago

Thank you @swingerman !