-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
For newly added TS0601_soil TuYa Soil sensor: expose soil_moisture instead of humidity #15946
Comments
@kris969 @cloudbr34k84 what do you think? |
|
I have no idea yet on how it's reported on other soil sensors... The exact is description is: A suggestion should be to expose both soil_moisture and humidity with, of course, the same report value. This way, everyone will be ok, and existing usages will not be impacted. |
My point/idea (reason for this report) is that reported data should be consistent. Again, air humidity and humidity of soil (soil_moisture) are not the same. You may have wet soil and dry air or another way around... This sensor report only one of these two values, which is the humidity of the soil. |
@f0gster : I agree! |
Done! Changes will be available in the dev branch in a few hours from now. (https://www.zigbee2mqtt.io/advanced/more/switch-to-dev-branch.html) |
Hi,
A rapid check with the dev branch reveale an issue.
no soil_moisture was reported!
I had to pair my sensors again to have it working fine.
This is just a comment for everyone. If you started with previous z2m
version, you will need to delete your sensor from z2m then repair it
again.
Le vendredi 06 janvier 2023 à 00:50 -0800, Koen Kanters a écrit :
… Done!
Changes will be available in the dev branch in a few hours from now.
(https://www.zigbee2mqtt.io/advanced/more/switch-to-dev-branch.html)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I use this channel to address this comment to soil sensor'users. here are information reportes by z2m: |
Hello, I got the new version of this device, they said the battery issue is fixed with this version. After switching z2m to dev branch I was able to pair it succesfully, however, the states are not reporting, only lqi, the rest says: null |
After a few days, it fixed it self! |
What happened?
1st of, thanks for adding TS0601_soil TuYa Soil sensor.
I have the device, so I played with an external converter before 1.29. It was awfully similar to your added code, apart of one thing: I'm suggesting changing humidity, that you are currently exposing, to soil_moisture.
All existing soil sensors are exposing soil_moisture (some also humidity on top of that).
TuYa Soil sensor is clearly not sending air humidity - that is easy to check. When I have a sensor in the air (not in soil), for example just near me, it will send 0. Obviously, air humidity is not 0 (all other sensors exposing humidity will show something different).
Hope it makes sense to you.
What did you expect to happen?
Expose soil_moisture
How to reproduce it (minimal and precise)
No response
Zigbee2MQTT version
1.29.0
Adapter firmware version
20221102
Adapter
Sonoff Zigbee 3.0 USB Dongle Plus CC2652P
Debug log
No response
The text was updated successfully, but these errors were encountered: