Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

New feature proposal: temperature offset #10

Open
kozerskil opened this issue Jan 2, 2020 · 5 comments
Open

New feature proposal: temperature offset #10

kozerskil opened this issue Jan 2, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@kozerskil
Copy link

I have 2 thermostats in 1 room, however one of them should be set to lower temperature. It would be nice to have a possibility to configure the offset for all of the thermostats.

@daenny
Copy link
Owner

daenny commented Jan 7, 2020

How would you then like to have the setpoint temperature shown? Should it be the average, or the adjusted average?

@daenny
Copy link
Owner

daenny commented Jan 7, 2020

Also this would mean that we need to change the method how the calls are made. At the moment only one service call is made to set the temperature etc, with the list of all entities, if you have offsets you would need to make one service call per entity.

@kozerskil
Copy link
Author

I think there should be one "master", eg. first entity, and the temperature shown is master temperature. Regarding many call, I don't think it is an issue: how many thermostats there can be in single group? And there might be some logic, grouping all thermostats with the same temperature, and one call made per subgroup. Not sure if it's not overcomplicated, but for me it would be valuable.

@daenny daenny added the enhancement New feature or request label Apr 3, 2020
@blademckain
Copy link
Contributor

blademckain commented Jun 3, 2020

the master slave functionality could be easily solved if you could choose whether to fill the component automatically or manually. Example of code:

climate:
  - platform: climate_group
    name: 'Climate Friendly Name'
    temperature_unit: C  
    entities:
    - climate.clima1
    - climate.clima2
    mode: master or group
    load: automatic or manual
    preset:
      -name: home
        target_temperature_low: 20
        target_temperature_high: 25
      -name: away
       target_temperature_low: 16
       target_temperature_high: 18
     hvac_modes: HVAC_MODE_OFF, HVAC_MODE_HEAT
     fan_modes: FAN_ON, FAN_OFF
....
   *other properties from https://developers.home-assistant.io/docs/core/entity/climate/
....

Mode
master: sends the changes to the other climates but does not receive them (except the current temperature)
group: that is, it behaves as it is now

Load:
automatic: takes all the characteristics of the other climates (that is, it behaves as it is now)
manual: creates a climate component with only those characteristics (obviously they must be compatible with the other climates)

@LucidityCrash
Copy link

Also this would mean that we need to change the method how the calls are made. At the moment only one service call is made to set the temperature etc, with the list of all entities, if you have offsets you would need to make one service call per entity.

You know this might be a better idea anyway ... if you have a lot of group members it can time out ... I use this with the Wiser integration and any more than 4 or 5 in a group tends to cause issues.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants