Skip to content
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

Gardena Sensors not taken into consideration #456

Closed
1 task done
w1Ngx opened this issue Jul 29, 2024 · 8 comments
Closed
1 task done

Gardena Sensors not taken into consideration #456

w1Ngx opened this issue Jul 29, 2024 · 8 comments

Comments

@w1Ngx
Copy link

w1Ngx commented Jul 29, 2024

What happened?

Gardena Humidity Sensor shows 15%, still 0 duration for the zone calculated

How to reproduce

click calculate

Relevant log output

none - more a question on how i can check if the sensor is ignored?

Which version are you running?

2024.7.8

Diagnostics file

  • I have attached a diagnostics file

Additional information

config_entry-smart_irrigation-3449ac96973de5ce532ac215b3e02eb6.json

@w1Ngx w1Ngx added the bug Something isn't working label Jul 29, 2024
@jeroenterheerdt
Copy link
Owner

Did you wire it up in the sensor group? Then it it's not ignored. If you want all the gory details enable debug logging...

@jeroenterheerdt jeroenterheerdt added dev responded and removed bug Something isn't working labels Jul 29, 2024
@w1Ngx
Copy link
Author

w1Ngx commented Jul 30, 2024

Yes 2 sensors are maintained. Would there be an error message if it can’t read them?
IMG_2500

Today it’s 31 degrees in Germany, humidity sensor is at 10%, still 0 irrigation calculated for today.

@jeroenterheerdt
Copy link
Owner

It should log an error, but turn on debug logging to find if anyone is going wrong. Did you look at the logs at all?

@w1Ngx
Copy link
Author

w1Ngx commented Jul 31, 2024

home-assistant_smart_irrigation_2024-07-31T18-08-24.008Z.log

yes i checked the logs, there was nothing related to the sensors.
last night it just gave an error for the automation template for some reason, so it didnt trigger the valve to be opened

@jeroenterheerdt jeroenterheerdt added bug Something isn't working and removed dev responded labels Aug 2, 2024
@Dr-ead
Copy link

Dr-ead commented Aug 18, 2024

As far as I understand, you need to use an air humidity not a soil moisture sensor! As in your screenshot, it is a soil moisture/humidity sensor. Smart irrigation has no input for soil moisture sensors.
So you can argue if the calculation is correct, when your soil moisture is 15% and there is no irrigation, if everything else is setup correctly and works properly.

But I observe a similar issue here using smart irrigation to irrigate my flower garden. I think the calculation is more or less correct. But if you look on my soil moisture sensors, it doesn't fit. So I think I must play arround with the parameters of maximum bucket size and also apply a real soil moisture start level, from which smart irrigation calculates on. Especially if you let the system not irrigate for e.g. 5 days and there was one short but heavy rain shower on one day otherwise it was hot and sunny, the calculation doesn't fit anymore, as the soil moisture is reducing extremly over these days (and that you can see on the flowers too) but the calculated bucket size is still positive. So in my opinion the soil is not saving the water as assumed in my case. Maybe it is easier, if you trust the soil moisture sensors, to start your irrigation system below xx% and stop it above yy%, I think to realize it like that, too, but currently I'm not so convinced of the soil moisture sensors I use or lets say I do not know at which values I need or not need to irrigate. As I read it in the discussions, another important point; I do not have any sudden peaks on my soil moisture sensors with their position and my irrigation system.

@w1Ngx
Copy link
Author

w1Ngx commented Aug 18, 2024

The humidity and temperature are for air, not soil? isn’t soil measuring the most accurate indicator for water needs?

If it’s like that i assume weather service is a better choice and the soil measuring will be irrelevant, or is there any good use of it in this integration?

@jeroenterheerdt
Copy link
Owner

jeroenterheerdt commented Aug 20, 2024

yes, humidity and temperature are for air, not soil.
Soil sensors are not a good way to measure watering needs: they spike and drop off real quick (just put some water on your soil sensor and observe the data). Also, they are point solutions (so unless you want to invest in putting soil sensors every 2 meters they are useless). The industry standard on which this integration is based does not use soil sensors at all, for these reasons.

So to call this integration useless or irrelevant is a bit of a stretch. In fact, it's the opposite: soil humidity sensors are useless.

Anyway as a way to make sure you don't have to invest in anything: no sensors, not weather station... you can use this integration with just a weather service. However, the better way of doing this is get a weather station and use that data.

@jeroenterheerdt jeroenterheerdt added dev responded and removed bug Something isn't working labels Aug 22, 2024
Copy link

Stale issue message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants