Expand support for Aqara E1 Thermostat

[jymdman]

This PR expands the support for Aqara E1 Thermostat.

Known issues:

• Can't turn on/off device with standard thermostat control. (With cluster command you can.)
• OTA update works, but reports wrong firmware version in UI as Aqara don't use standard way of reporting this. You need to issue a cluster command to start update.

[coveralls]

Pull Request Test Coverage Report for Build 3934025323

• 23 of 79 (29.11%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.5%) to 83.035%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
zhaquirks/xiaomi/aqara/thermostat_agl001.py	23	79	29.11%

Totals
Change from base Build 3930807394:	-0.5%
Covered Lines:	6813
Relevant Lines:	8205

---

💛 - Coveralls

[codecov-commenter]

Codecov Report

Base: 83.55% // Head: 83.03% // Decreases project coverage by -0.51% ⚠️

Coverage data is based on head (e1b5ff5) compared to base (1891e3f).
Patch coverage: 29.11% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #2109      +/-   ##
==========================================
- Coverage   83.55%   83.03%   -0.52%     
==========================================
  Files         252      252              
  Lines        8128     8205      +77     
==========================================
+ Hits         6791     6813      +22     
- Misses       1337     1392      +55     

Impacted Files	Coverage Δ
zhaquirks/xiaomi/aqara/thermostat_agl001.py	40.40% <29.11%> (-41.42%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[TheJulianJES]

What is the cluster command to turn on/off the thermostat?
We could either add an entity for that directly in HA Core (ZHA) or possibly just make a fake cluster in zha-quirks (so ZHA creates the entity).

[jymdman]

On/off is system_mode (0x0271). 0 for off, 1 for on.

service: zha.set_zigbee_cluster_attribute
data:
ieee: 54:ef:44:10:00:57:b5:c6
endpoint_id: 1
cluster_type: in
cluster_id: 64704
attribute: 625
value: 0

[dmulcahey]

@puddly you have this device right? I thought it worked OOTB for standard functions?

[puddly]

Yeah, it shows up as a thermostat for me without a quirk. Perhaps there are multiple firmware revisions?

It reports its current temperature. Adjusting the setpoint in the frontend causes the plunger to move appropriately and for the display's temperature setting to change.

Mine is lumi.airrtc.agl001.

• hw_version=1
• date_code=Jun 7 2022
• sw_build_id=0.0.0_0025
• app_version=25

There is no OTA cluster.

[renaudallard]

It looks like this is working fine here. I haven't tested OTA though.

[Sirse]

Working fine, but can't test OTA (already latest firmware).

[TheJulianJES]

@puddly

There is no OTA cluster.

Looking at the quirk, it seems like the Aqara E1 thermostat has a "hidden" OTA cluster.

[TheJulianJES]

I know this isn't great, but would you be willing to split the aqaraHeader, _float_to_hex, write_attributes, and "fake address" stuff into another PR? Those might need tests and are all a bit more complicated.

Splitting that off into another PR would keep this one relatively simple, so we can merge it (it would mostly just be the added custom Xiaomi attributes). If the attributes are properly present then, we can start creating sensors in ZHA (HA Core) for them.

[koehleru]

@TheJulianJES
i am also interested in progress of this integration. So i created a PR #2237 with the simple part.

[jymdman]

Since I moved my system to z2m I really can't test any changes anymore.
But please use my code if useful for the implementation.

[TheJulianJES]

Alright, thanks.
I also bought a thermostat to test this.

[TheJulianJES]

@jymdman Ah, one last thing. Could you send a screenshot of how Z2M shows the thermostat in HA?
(I'm wondering about presets, window_detection, away_preset_temperature, and so on)

[jymdman]

@TheJulianJES Sure! Attaching a screenshot...

[github-actions]

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

[okainov]

Any updates on this one? @TheJulianJES @jymdman ?