-
Notifications
You must be signed in to change notification settings - Fork 862
Add Support for HOBEIAN (Tuya) soil sensor ZG-303Z
and Tuya soil sensor _TZE200_npj9bug3
#4160
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
base: dev
Are you sure you want to change the base?
Conversation
dc3acba
to
461f114
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #4160 +/- ##
=======================================
Coverage 91.28% 91.28%
=======================================
Files 339 339
Lines 10967 10971 +4
=======================================
+ Hits 10011 10015 +4
Misses 956 956 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
87d976f
to
88d4a32
Compare
…0_npj9bug3` variant
02a3438
to
8346cd3
Compare
To avoid an accidental merge of this PR in its current state: As written in the referenced issues, this PR does (for now) add new sensor information and adjustment settings (calibration) but does not (yet) tackle a "soil moisture/air humidity is sharing values" bug which I reported in #4122. Makuser is surely just a little step away from fixing these small annoyancies too. So I hope this post here does not disencourage you to continue the PR. |
Work for humidity on sol on hard copy on core, when we merge this ? |
@sandros06 so "dry" and "wet" is not mixed on your installation? Here it is marked as "wet" if there is only some percents of moisture - but "dry" if I just added fresh water. Means "please give water" alarms won't work the way people expect (you need to check for the opposite value). |
Hmm, you're right! I missed this part and it's not working very well. |
Works fine here with device ZG-303Z.
|
I tried this out on my ZG-303Z. It seems to be working, I got a whole set of configuration controls, and the moisture value. However I also noticed the dry/wet binary sensor is mixed up. I watered the plant and moisture reports 100%, while the binary sensor is still "dry". |
Proposed change
Add Support for HOBEIAN (Tuya) soil sensor
ZG-303Z
and Tuya soil sensor_TZE200_npj9bug3
Additional information
Fixes #4144, #4122 (comment)
Checklist
pre-commit
checks pass / the code has been formatted using Black