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

Group mode not working #68

Closed
2 tasks done
bruvv opened this issue Dec 4, 2021 · 6 comments
Closed
2 tasks done

Group mode not working #68

bruvv opened this issue Dec 4, 2021 · 6 comments
Labels
help wanted Extra attention is needed under investigation If its not clear whats the problem

Comments

@bruvv
Copy link
Contributor

bruvv commented Dec 4, 2021

TL:DR;

Grouping thermostats isn't working.

Prerequisites

  • Model name of your Devices from Zigbee2Mqtt: Tuya TS601
  • Output from Home Assistent Developer Tools state e.g.
  - heat
  - auto
  - 'off'
min_temp: 5
max_temp: 35
target_temp_step: 0.5
preset_modes:
  - none
  - away
  - schedule
  - manual
  - boost
  - complex
  - comfort
  - eco
current_temperature: 19
temperature: 7
preset_mode: manual
auto_lock: MANUAL
away_mode: 'OFF'
away_preset_days: 1
away_preset_temperature: 5
battery_low: false
boost_time: 300
child_lock: UNLOCK
comfort_temperature: 22
current_heating_setpoint: 7
device:
  applicationVersion: 85
  dateCode: ''
  friendlyName: '0x5c0272fffed9dcb0'
  hardwareVersion: 1
  ieeeAddr: '0x5c0272fffed9dcb0'
  manufacturerID: 4098
  manufacturerName: _TZE200_ckud7u2l
  model: TS0601_thermostat
  networkAddress: 35689
  powerSource: Battery
  stackVersion: 0
  type: EndDevice
  zclVersion: 3
eco_temperature: 20
force: close
holidays:
  - hour: 6
    minute: 0
    temperature: 20
  - hour: 8
    minute: 0
    temperature: 15
  - hour: 11
    minute: 30
    temperature: 15
  - hour: 12
    minute: 30
    temperature: 15
  - hour: 17
    minute: 30
    temperature: 20
  - hour: 22
    minute: 0
    temperature: 15
holidays_schedule: 06:00/20°C 08:00/15°C 11:30/15°C 12:30/15°C 17:30/20°C 22:00/15°C
linkquality: 66
local_temperature: 19
local_temperature_calibration: 0.5
max_temperature: 25
min_temperature: 5
position: 0
preset: manual
system_mode: 'off'
update:
  state: available
update_available: true
valve_detection: null
week: 5+2
window_detection: 'OFF'
window_detection_params:
  minutes: 0
  temperature: 109
workdays:
  - hour: 6
    minute: 0
    temperature: 20
  - hour: 8
    minute: 0
    temperature: 15
  - hour: 11
    minute: 30
    temperature: 15
  - hour: 12
    minute: 30
    temperature: 15
  - hour: 17
    minute: 30
    temperature: 20
  - hour: 22
    minute: 0
    temperature: 15
workdays_schedule: 06:00/20°C 08:00/15°C 11:30/15°C 12:30/15°C 17:30/20°C 22:00/15°C
friendly_name: badkamer_thermostaat
supported_features: 17

Description

I have multiple thermostats in my different rooms. Since I want to adjust 2 at the same time I use this addon: https://github.com/daenny/climate_group

I have setup groups us followed:

  - platform: climate_group
    name: 'Woonkamer Thermostaat AI'
    temperature_unit: C  # default to celsius, 'C' or 'F'
    entities:
    - climate.living_room_1_ai
    - climate.living_room_2_ai

And this is the ai setup:

  - platform: ai_thermostat
    name: "Living Room 1 AI"
    unique_id: 435345346 # Its important for Google Home
    thermostat: climate.woonkamer_thermo_1
    temperature_sensor: sensor.woonkamer_temp_temperature
    #window_sensors: group.office_windows #if this is not set, the window open detection is off
    weather: weather.openweathermap #if this is set, the outdoor_sensor is ignored, remove the outdoor_sensor config!
    #outdoor_sensor: sensor.outdoor_temperature #if you want to use it, remove the weather config!
    off_temperature: 18
    #window_off_delay: 0
    valve_maintenance: false
    night_temp: 5
    night_start: '21:30'
    night_end: '08:00'
  - platform: ai_thermostat
    name: "Living Room 2 AI"
    unique_id: 435345341 # Its important for Google Home
    thermostat: climate.woonkamer_thermo_2
    temperature_sensor: sensor.woonkamer_temp_temperature
    #window_sensors: group.office_windows #if this is not set, the window open detection is off
    weather: weather.openweathermap #if this is set, the outdoor_sensor is ignored, remove the outdoor_sensor config!
    #outdoor_sensor: sensor.outdoor_temperature #if you want to use it, remove the weather config!
    off_temperature: 18
    #window_off_delay: 0
    valve_maintenance: false
    night_temp: 5
    night_start: '21:30'
    night_end: '08:00'

Now only the first one is being changed. Is there perhaps an option to use ai_thermostat to group different thermostats?

Steps to Reproduce

  1. Group different climate trvs
  2. Only 1 will change
  3. you will get cold

Expected behavior:

All of them change.

Actual behavior:

only 1 changes

Versions

Home Assistant: 2021.11.5
Zigbee2mqtt: 1.22.1-1

@KartoffelToby
Copy link
Owner

i didn't tried it. but i think about it.

because the different TRV have different local temperaturs its not good to group them.

but. it should be working if you create for each of your TRV a ai TRV and groupe them.

@KartoffelToby
Copy link
Owner

#22

@bruvv
Copy link
Contributor Author

bruvv commented Dec 5, 2021

Also to update here: as mentioned in the issue: grouping is not working.

@KartoffelToby
Copy link
Owner

okay if the other way of grouping dosn't work i must debug it. i'll build up a test Bench with two Spirits soon.

@KartoffelToby KartoffelToby added under investigation If its not clear whats the problem help wanted Extra attention is needed labels Dec 5, 2021
@KartoffelToby
Copy link
Owner

@bruvv I test it with my Spirits and Moes, and it worked in that way i described you, to make it clear i have a basic config demo:

- platform: ai_thermostat
  name: Ai - TRV - Office - 1
  thermostat: climate.real_trv_office_1
  temperature_sensor: sensor.temperatur_office_temperature
  window_sensors: group.office_windows
  weather: weather.home
  off_temperature: 20
  unique_id: 1
- platform: ai_thermostat
  name: Ai - TRV - Office - 2
  thermostat: climate.real_trv_office_2
  temperature_sensor: sensor.temperatur_office_temperature
  window_sensors: group.office_windows
  weather: weather.home
  off_temperature: 20
  unique_id: 2
- platform: climate_group
  name: "TRV - Office"
  temperature_unit: C
  entities:
    - climate.ai_trv_office_1
    - climate.ai_trv_office_2

@bruvv
Copy link
Contributor Author

bruvv commented Dec 6, 2021

For some weird reason it was all messy with my unique_ids that I didn't bother changing since I do not use google home. Removing them like in your example fixed it for me.
Added a PR to elaborate on this: #77

@bruvv bruvv closed this as completed Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed under investigation If its not clear whats the problem
Projects
None yet
Development

No branches or pull requests

2 participants