Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disabled thermostat turns off switch when manually turned on. #136

Closed
mtausk opened this issue Mar 2, 2023 · 8 comments
Closed

Disabled thermostat turns off switch when manually turned on. #136

mtausk opened this issue Mar 2, 2023 · 8 comments

Comments

@mtausk
Copy link

mtausk commented Mar 2, 2023

Describe the bug
When the termostat is turned off and I manually turn on heater, the thermostat turns off the heater after it discovers the entity is on.

Similar issue is described here:
https://community.home-assistant.io/t/random-service-homeassistant-turn-off/281671/18

To Reproduce
Steps to reproduce the behavior:

  • stop thermostat (aka disable it)
  • turn on the switch (the same one that is controlled by thermostat)
  • wait few seconds
  • thermostat becomes "alive" and turns the button off for you even if you are not very happy with that decision.

Expected behavior

When I turn off thermostat, it should stay disabled and allow other possibilities to turn the heater/ac switch on without further interference (eg. automations or manual switch on).

Expected behavior would be when I turn on the thermostat, it turns off the entity and after successful entity switch-off, it should stop watching switch status of the entity, ie. thermostat should "disable" itself or at least stop meddling with the switch...

This is written to the log:
2023-03-02 17:30:46.041 INFO (MainThread) [custom_components.smart_thermostat.climate] climate.konvektor_detska1_pid: Turning OFF switch.konvektor_detska_1

The logbook says:
"Turned off by service homeassistant.turn_off"

Screenshots
obrázok

Version
Current / 2023.2.1

Desktop (please complete the following information):
Not applicable

Smartphone (please complete the following information):
Not applicable

Thank you!!!

@ScratMan
Copy link
Owner

ScratMan commented Mar 2, 2023

This has been added on purpose. An issue has been raised in the past requesting the switch to be forced synced to the thermostat, otherwise the heater could be locked in on state.

As a workaround, you may use a Boolean helper as heater entity in the thermostat with an automation to turn on or off your real heater when helper's state toggles. Then by disabling the thermostat, the helper will remain off, the automation will not be triggered and you'll be able to set the heater on or off manually.

@mtausk
Copy link
Author

mtausk commented Mar 3, 2023

First of all, thank you for your answer.

Workarounds are there to work-around an existing problem. I have lots of integrations in my homeassistant, but no workaround since everything is working independently and as designed.

I kindly ask you to reevaluate this issue. What are the odds and possibilities that the switch would remain in on state when you turn off the thermostat?
I believe that it should be possible to disable everything that has any influence on entities, just as you can disable automations, for instance - otherwise it can interfere with other intentions and have other consequences. Can you point me to the past problem due to which this is now forced?

So, with the past problem (requesting the switch to be forced synced to the thermostat) another problem has been introduced and so far a workaround is proposed. However, what needs to be reevaluated is this:

  • Can the forced syncing be redesigned?
  • In which circumstances could that problematic state occur? Outages and reboots only?
  • Usually entities are configurable in regards to their state after outage (previous, off, on), which means this option to force-sync is redundant in case of outages. It is better to set device's state after reboot to "off".
  • Can you add some option as a solution (eg. force-sync)?
  • I understand this is a security measure but it should not be forced to all users and should be configurable.

Please, let's think about how this can be solved for both sides.

@ScratMan
Copy link
Owner

ScratMan commented Mar 3, 2023

Understand, I'll look for a way to add a parameter to force sync heater off when thermostat is disabled only if required.

@ulicskad
Copy link

This feature tricked me :) I have got 4 PID climate for my 4 room, but the heater switch are common. Only 1 PID can be turned on, others switches off automaticly. In the last 2 years it was working well, but after the last update the heater keeps turning off every 30 seconds. It was a few cold days while i figured out what happened. (Also my floor heating pid worked well...). If this feature could be configurable, that would be great. Also great work with the thermostat, i like it very much, and saves me a lot of gas price.
D

@AnduriI
Copy link

AnduriI commented Jul 5, 2023

@ScratMan is there now the option to disable the force_OFF behaviour? As described in #86 I would really like to have PID for heating on room level, but also have the option to cool without restart of HA. The easiest way would be to manually set the valve (or with an automation) where cooling is wanted. I don't need PID for cooling as my heatpump is doing that according to the outside temperature for the complete house.

@ScratMan
Copy link
Owner

ScratMan commented Jul 6, 2023

For #86 , it's work in progress, I can switch the mode from heat to cool via a service, but the heater entity still acts as a pure heater, enabling when temperature is below setpoint even in cooling mode.

@ScratMan
Copy link
Owner

Hi, v2023.7.0-beta1 is available with the new force_off_state parameter.
You can switch to this pre-release by enabling "beta versions" in HACS download screen.
I will make the final release if the pre-release is confirmed to work well. Thanks.

@AnduriI
Copy link

AnduriI commented Aug 30, 2023

Thank you, that fix works. I think this can be closed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants