Aqara E1 thermostat external temperature sensor

[mb1248]

Proposed change

Support for external temperature sensor.

Additional information

This is a copy of the not merged part from #2109. Discussion starts under #2687

Checklist

• The changes are tested and work correctly
• pre-commit checks pass / the code has been formatted using Black
• Tests have been added to verify that the new code works

[codecov]

Codecov Report

Attention: Patch coverage is 24.19355% with 47 lines in your changes missing coverage. Please review.

Project coverage is 87.17%. Comparing base (04ac526) to head (7f30c58).
Report is 317 commits behind head on dev.

Files with missing lines	Patch %	Lines
zhaquirks/xiaomi/aqara/thermostat_agl001.py	24.19%	47 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #2802      +/-   ##
==========================================
- Coverage   87.61%   87.17%   -0.44%     
==========================================
  Files         295      295              
  Lines        9032     9094      +62     
==========================================
+ Hits         7913     7928      +15     
- Misses       1119     1166      +47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[fzob]

What is the status here? I could help implement or test, as I would also like to use this feature in the future :-)

[mb1248]

Hi @fzob, feel free. The new cluster appears in the ZHA device tab and I can send command to it. But if I read them they are not changed. My guess is, that the message is not correct, but I didn‘t find the time to debug it.

[fzob]

Indents are wrong on line 403-567. So custom attributes weren't written. After this fix setting sensor and sensor_temp works!

[mb1248]

@fzob I fixed it. Coverage could gave a hint on that 🤦 . Anyway, thanks for taking a look. Now, it also works for me.

[Patrick-DE]

@mb1248 I have added this quirk to my ZHA, now I am asking myself a very stupid question.
Is it correct that I am defining the external sensor by going to Manage Zigbee Device -> Cluster (In) -> Sensor, and write the IEEE of my temp sensor via the value?
Also, does it automatically use the external sensor temperature entity or do I still need an automation to push the temp entity of an external sensor into the SENSOR_TEMP entity?

[mb1248]

Hi @Patrick-DE, you need to add an automation. The sensor value is either internal (Default/0) or external, meaning that the internal temperature sensor is either used or not. If an external sensor is selected, the temperature value has to be set manually. This works similarly to how the Aqara hub operates. Note that, due to compatibility with Z2M, the temperature value is in centi-degrees (c°C), so you have to multiply it by 100. (This seems awkward, and I'm considering changing it.) I also read that the sensor type is reset; however, I didn't observe this in my test (around 2 hours). Therefore, the automation for Z2M, first, set the sensor type and then the temperature value.

[DamGranger]

Hi, I'm trying to add an automation based on this quirk. Unfortunately I'm stuck when trying to pass the value of the external sensor...

That's what I'm trying to do and it ends up with 0° in the TRV current temperature :

service: zha.set_zigbee_cluster_attribute
data:
  cluster_type: in
  ieee: 54:ef:44:10:00:73:cb:f6
  endpoint_id: 1
  cluster_id: 64704
  attribute: 5010
  value: "{{ states('sensor.cl_chp_temperature')|float|round(1) * 100 }}"

It works fine when putting an hardcoded value like 2100, though...

Would you have any idea?

[tr4nt0r]

you have to pass an integer, after the multiplication add |int and it should work

[DamGranger]

you have to pass an integer, after the multiplication add |int and it should work

service: zha.set_zigbee_cluster_attribute
data:
  cluster_type: in
  ieee: 54:ef:44:10:00:73:cb:f6
  endpoint_id: 1
  cluster_id: 64704
  attribute: 5010
  value: "{{ states('sensor.cl_chp_temperature')|float|round(1) * 100|int }}"

Nevermind, I'm stoopid...

That works with the right parenthesis

service: zha.set_zigbee_cluster_attribute
data:
  cluster_type: in
  ieee: 54:ef:44:10:00:73:cb:f6
  endpoint_id: 1
  cluster_id: 64704
  attribute: 5010
  value: "{{ (states('sensor.cl_chp_temperature')|float|round(1) * 100)|int }}"

Thanks

[DamGranger]

I someone wants to try your quirk that's how I made it work as a whole for one TRV, once I migrate the other 9, I'll just add some triggers and template up the ieee (or duplicate the actions if I can't generate the ieee in a template) :

alias: TH Mise à jour température vannes
description: Spécifique aux vannes Aqara SRTS-A01
trigger:
  - platform: state
    entity_id:
      - sensor.cl_chp_temperature
    id: parents
condition: []
action:
  - choose:
      - conditions:
          - condition: trigger
            id:
              - parents
            enabled: false
          - condition: template
            value_template: >-
              {{ (state_attr('climate.th_chp_thermostat',
              'current_temperature')|float -
              states('sensor.cl_chp_temperature')|float)|abs >= 0.1 }}
        sequence:
          - variables:
              sensor_temperature: >-
                {{ (states('sensor.cl_chp_temperature')|float|round(1) *
                100)|int }}
          - alias: Définir le capteur externe
            service: zha.set_zigbee_cluster_attribute
            data:
              cluster_type: in
              ieee: 54:ef:44:10:00:73:cb:f6
              endpoint_id: 1
              cluster_id: 64704
              attribute: 638
              value: "1"
            enabled: true
          - delay:
              hours: 0
              minutes: 0
              seconds: 3
              milliseconds: 0
            enabled: true
          - service: zha.set_zigbee_cluster_attribute
            data:
              cluster_type: in
              ieee: 54:ef:44:10:00:73:cb:f6
              endpoint_id: 1
              cluster_id: 64704
              attribute: 5010
              value: "{{ sensor_temperature }}"
            alias: Passer la température du capteur externe
mode: queued
max: 10

[StrandmonYellow]

Unfortunately this does not work for me. Turning on the external sensor command does get completed by homeassistant, but the entity state keeps saying 'off'. Additionally, i get an error when the temperature needs to be sent.

Can you maybe make a blueprint for this?

Update:
Solved it already. First had to add the custom quirk ofcourse :)

[Antoninj]

Hi. Is there any chance this gets merged soon? Is this blocked only by testing? If that’s the case, I can maybe help.

[mb1248]

@Antoninj I had no time to work on the test. The quirk is working. So only the testing is missing. Would be nice if you could help there.

[cpaius]

Thanks, it works for me too!

[hraschan]

It seem to work for a few people here. When can the PR get resolved? Is there still stuff to do? Would love to use the feature. :) I also tried the custom quirk and @DamGranger automation. Works like a charm

[Krzemq]

How is it going? Will it be working? It's last thing i have on z2m :<

[sroettering]

Would love this feature as well :)

[rolfsan]

Yes It would be nice👍

[DPellone]

Any update on this? It would be a nice feature to add 👍

[StrandmonYellow]

Is there any update on this?

[m00nyONE]

Can this be merged please? I really want to switch from z2m to zha and this is the only reason i cen't

[codesalatdev]

Can this be merged please? I really want to switch from z2m to zha and this is the only reason i cen't

The situation seems to be still the same, with tests missing in the PR.

[hraschan]

Can somebody add them? I thing that should be quick thing to do.

[Antoninj]

Update: I've opened a new PR containing changes from this PR + the missing tests