core icon indicating copy to clipboard operation
core copied to clipboard

climate.set_temperature requires too much information for heat/cool thermostats

Open ve6rah opened this issue 4 years ago • 27 comments

The problem

When calling climate.set_temperature, you are required to send all of hvac_mode, target_temp_low, target_temp_high even when you only want to set one of those three values. This generates unnecessary traffic on low bandwidth modes (e.g. z-wave), requires additional templating in automations when trying to leave one of the other values at "whatever it was before", and doesn't play nice with some thermostats (e.g. CT32)

Considering that manual temperature changes through lovelace thermostat cards only send the one value you're trying to change, and not all 3, it seems that there is no technical reason to force the user to enter the other information if they do not want it changed, and several problems would be solved by removing the requirement.

What is version of Home Assistant Core has the issue?

core-2021.3.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

climate

Link to integration documentation on our website

https://www.home-assistant.io/integrations/climate

Example YAML snippet

service: climate.set_temperature
target:
  device_id: 621ca738187f2fb3809f05937a0d63b9
data:
  target_temp_low: 21.5

Anything in the logs that might be useful for us?

Failed to call service climate/set_temperature. some but not all values in the same group of inclusion 'temperature' @data[<temperature>]

This would fix several open bugs, while at the same time making automating things much easier.

ve6rah avatar Mar 08 '21 16:03 ve6rah

climate documentation climate source (message by IssueLinks)

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jun 06 '21 17:06 github-actions[bot]

Issue still exists with latest versions

ve6rah avatar Jun 06 '21 18:06 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Sep 04 '21 20:09 github-actions[bot]

Issue still exists in latest versions

ve6rah avatar Sep 04 '21 22:09 ve6rah

Still exists in 2021.10.4. Additional findings. Service Viessmann ViCare: Set vicare mode requires to pick an entity, but the system does not offer any.

frank-dusign avatar Oct 27 '21 11:10 frank-dusign

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jan 25 '22 12:01 github-actions[bot]

Issue still exists in latest versions

ve6rah avatar Jan 25 '22 16:01 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Apr 30 '22 12:04 github-actions[bot]

Issue still exists in latest versions

ve6rah avatar Apr 30 '22 13:04 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Aug 03 '22 22:08 github-actions[bot]

Issue still exists in latest versions

ve6rah avatar Aug 03 '22 22:08 ve6rah

Issue still exists on the latest version

sresam89 avatar Sep 23 '22 19:09 sresam89

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Issue still exists in latest versions

ve6rah avatar Jan 15 '23 23:01 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Issue still exists in latest versions

ve6rah avatar Apr 16 '23 00:04 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Issue still exists in latest versions

ve6rah avatar Jul 15 '23 01:07 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Issue still exists in latest versions

ve6rah avatar Oct 13 '23 06:10 ve6rah

@ve6rah I love your perseverance to keeping this issue alive. Looking into it it seems to be integration specific. For instance in the generic_thermostat only a temperature or hvac_mode can be set for climate.set_temperature and none of them are mandatory. I think you should direct your issue to the right integration which in your case seems to be the radiotherm integration.

rrooggiieerr avatar Nov 04 '23 04:11 rrooggiieerr

@ve6rah I love your perseverance to keeping this issue alive. Looking into it it seems to be integration specific. For instance in the generic_thermostat only a temperature or hvac_mode can be set for climate.set_temperature and none of them are mandatory. I think you should direct your issue to the right integration which in your case seems to be the radiotherm integration.

This was originally under the ancient z-wave integration, however at that time it was pointed out that it was not integration specific, and was very much an issue with core. I've double checked, this is not integration specific. Currently I am using this through zwave JS UI, however I've also tried through the last three z wave integrations, and they were all the same. After investigating it further it is very much part of climate.set_temperature. I would bet that if your HVAC mode was heat/cool, it would force you to set both a high and low temperature, with no ability to set only the one temperature you are interested in.

ve6rah avatar Nov 04 '23 14:11 ve6rah

Yes, you're right, could very well be it's the HVAC mode of my generic_thermostat that is limiting the options

rrooggiieerr avatar Nov 10 '23 07:11 rrooggiieerr

As I said in this issue, I have proven that it is not your thermostat limiting it. It is home assistant that is artificially requiring more information than needs to be sent.

ve6rah avatar Nov 10 '23 12:11 ve6rah

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Issue still exists in latest versions

ve6rah avatar Feb 08 '24 17:02 ve6rah

I'm having the same issue.

AJolly avatar Feb 10 '24 00:02 AJolly

i dont have a fix for this issue but i think I figured out a reliable solution in node-red on home-assistant

TrueBankai416 avatar Mar 08 '24 22:03 TrueBankai416

temp_flow

flows.json

TrueBankai416 avatar Mar 09 '24 01:03 TrueBankai416