Add Device Support : Tuya TS0601 - TZE200_81isopgh - Garden Watering System

[simonbaudart]

closes #1556

One question: I used a "FlowMeasurement" to report a Volume of water consumed.
Is this right ? Or should I use something else ?

[coveralls]

Pull Request Test Coverage Report for Build 2352554713

• 21 of 21 (100.0%) changed or added relevant lines in 1 file are covered.
• 124 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+0.06%) to 80.376%

---

Files with Coverage Reduction	New Missed Lines	%
zhaquirks/tuya/mcu/init.py	2	98.06%
zhaquirks/tuya/init.py	122	71.98%

Totals
Change from base Build 2282780395:	0.06%
Covered Lines:	5468
Relevant Lines:	6803

---

💛 - Coveralls

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.37%. Comparing base (b802c1f) to head (42ecbf7).
Report is 609 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #1560      +/-   ##
==========================================
+ Coverage   80.31%   80.37%   +0.05%     
==========================================
  Files         227      228       +1     
  Lines        6773     6803      +30     
==========================================
+ Hits         5440     5468      +28     
- Misses       1333     1335       +2     

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

[rforro]

Code in this PR looks suspiciously similar #1450

[javicalle]

Just a little change to go.

[javicalle]

LGTM

[javicalle]

Thanks Simon. Pretty nice work.

[simonbaudart]

I had to make a change and remove the metering cluster because the SmartEnergy purpose is for Energy in kW and not in L
Error from HA : "WARNING (Recorder) [homeassistant.components.sensor.recorder] sensor.arrosage_smartenergy_metering_summation_delivered has unit L which is unsupported for device_class energy"

[dmulcahey]

Can you add back the metering and just add a constant attribute for unit of measurement

[dmulcahey]

I had to make a change and remove the metering cluster because the SmartEnergy purpose is for Energy in kW and not in L
Error from HA : "WARNING (Recorder) [homeassistant.components.sensor.recorder] sensor.arrosage_smartenergy_metering_summation_delivered has unit L which is unsupported for device_class energy"

Does the device measure total sum used or instant demand (flow rate in this case)

[simonbaudart]

I had to make a change and remove the metering cluster because the SmartEnergy purpose is for Energy in kW and not in L
Error from HA : "WARNING (Recorder) [homeassistant.components.sensor.recorder] sensor.arrosage_smartenergy_metering_summation_delivered has unit L which is unsupported for device_class energy"

Does the device measure total sum used or instant demand (flow rate in this case)

Should be the total sum (should because my device has hardware issue so I cannot test it, always at 0)

[simonbaudart]

Can you add back the metering and just add a constant attribute for unit of measurement

I reverted to to previous version that already has the constant attribute.
I think this is more a HA change to be done to manage the L.

[dmulcahey]

Yep I’ll look at that part

[Gilles2senior]

@simonbaudart , with all threads related to this Garden Watering Valve, I have difficulties identifying latest code and qwirk. Do you know where is the latest? From discussions, it looks to be finalized, but doesn't seem to have been integrated into any release of ZHA since the time it was closed. Any help would be welcome. Thanks