Adding Qiumi/Siterwell QM-GS361A-H04 TRV

[MattWestb]

Please close #738 then marge.

Mvh Mattias

[codecov-io]

Codecov Report

Merging #740 (7eb8a9c) into dev (4f14ef0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #740   +/-   ##
=======================================
  Coverage   82.95%   82.95%           
=======================================
  Files         176      176           
  Lines        3931     3931           
=======================================
  Hits         3261     3261           
  Misses        670      670           

Impacted Files	Coverage Δ
zhaquirks/tuya/valve.py	86.76% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f14ef0...7eb8a9c. Read the comment docs.

[coveralls]

Pull Request Test Coverage Report for Build 505842412

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 82.956%

---

Totals
Change from base Build 503912949:	0.0%
Covered Lines:	3261
Relevant Lines:	3931

---

💛 - Coveralls

[TheJulianJES]

Since the model seems to always be TS0601, would it make sense to also address this using #735?

[MattWestb]

Its sound good but i dont knowing how to do that :-((
Edit: Then also for the other devices type in the same quirk so all its being done the same way.

[tube0013]

Since the model seems to always be TS0601, would it make sense to also address this using #735?

I just got a pair of these - https://zigbee.blakadder.com/Moes_AM43-0_45_40-ES-EB.html

I have yet to join them to ZHA where I'll need to figure out a quirk but reading the z2m code they also appear to use the TS0601 model...... so a very different device with same model id.

[Adminiuga]

Fixes #738

[Adminiuga]

Since the model seems to always be TS0601, would it make sense to also address this using #735?

#735 is more of a "proof of concept" I have those two devices in prod, so was able to test it.
The same approach could be use for TS0601, just need to verify there are no other quirks with the exact same model and exact same "endpoint" signature (same input/output clusters, device type, endpoints etc)

If someone with the said valves could test it, then we could create another PR

[MattWestb]

Tuya is using the same zigbee module (and the same loaded firmware) for switches and all other devices that is talking "tuyanish" and is using it as one communicating channel for tunneling the tuya MCU that sending more or less MQTT commands thru the tunnel.
So only looking on the zigbee module hardware config (module name endpoints and clusters) and not manufacture data its very likela not working.

[Adminiuga]

I just got a pair of these - https://zigbee.blakadder.com/Moes_AM43-0_45_40-ES-EB.html

I wish blakadder was posting the device signature too. Can you join and post the device signature. That's exactly my hesitation, Tuya using the same model for a different device. Need to compare signatures.

[tube0013]

I just got a pair of these - https://zigbee.blakadder.com/Moes_AM43-0_45_40-ES-EB.html

I wish blakadder was posting the device signature too. Can you join and post the device signature. That's exactly my hesitation, Tuya using the same model for a different device. Need to compare signatures.

added via an issue.... #744

[blakadder]

I just got a pair of these - https://zigbee.blakadder.com/Moes_AM43-0_45_40-ES-EB.html

I wish blakadder was posting the device signature too. Can you join and post the device signature. That's exactly my hesitation, Tuya using the same model for a different device. Need to compare signatures.

I am now :D