letscontrolit / ESPEasy

Easy MultiSensor device based on ESP8266/ESP32
http://www.espeasy.com
Other
3.28k stars 2.22k forks source link

_P047_i2c-soil-moisture-sensor.ino higher moisture values #240

Closed moorjunge closed 7 years ago

moorjunge commented 7 years ago

Steps to reproduce

I had some sensors that give values over 800 on moisture, up to 1100. So i get an info "SoilMoisture: Bad Reading, resetting Sensor..." and nothing is send to the controller.

After change the line 174 at _P047_i2c-soil-moisture-sensor.ino to if (temperature>100 || temperature < -40 || moisture > 1100 || moisture < 1 || light > 65535 || light < 0) {

everything seems to be fine

Expected behavior

Tell us what should happen?

Actual behavior

Tell us what happens instead?

System configuration

Hardware:

Software or git version:

define ESP_PROJECT_PID 2016110801L

define VERSION 2

define BUILD 20000 // git version 2.0.0

define BUILD_NOTES " - Mega"

//build all plugins that are in test stadium

define PLUGIN_BUILD_TESTING

krikk commented 7 years ago

what reading do you get if you put the sensor in a glass of water?? and what do you get in dry air? ...1100 seems a bit high to me, never get more than 500, so the limit of 800 should be sufficient...

moorjunge commented 7 years ago

Here is a Screenshot with two sensors in moist soil, but i will also do some more tests at the Weekend.

Am 20.04.2017 10:46 nachm. schrieb "krikk" notifications@github.com:

what reading do you get if you put the sensor in a glass of water?? and what do you get in dry air? ...1100 seems a bit high to me, never get more than 500, so the limit of 800 should be sufficient...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/letscontrolit/ESPEasy/issues/240#issuecomment-295904532, or mute the thread https://github.com/notifications/unsubscribe-auth/ALRlRxO3EwGucKnyOprDBle4OvJzcqhXks5rx8QYgaJpZM4NDbEd .

krikk commented 7 years ago

please provide more info, otherwise this issue can be closed...

moorjunge commented 7 years ago

Hello,

it seems, that all my sensors are broken. I just ordered two new one.

So, i will close the issue.

Greetings Thiemo

2017-04-27 22:27 GMT+02:00 krikk notifications@github.com:

please provide more info, otherwise this issue can be closed...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/letscontrolit/ESPEasy/issues/240#issuecomment-297829700, or mute the thread https://github.com/notifications/unsubscribe-auth/ALRlR8PRrPw1HkRpQR6ZwQaooIZI7KLMks5r0PoogaJpZM4NDbEd .

--

Thiemo Rögener Forbacher Straße 4

22049 Hamburg

Festnetz 040 450 63950 Mobil 0170 540 4115

Email thiemo@roegener.info Homepage http://www.roegener.info

psy0rz commented 7 years ago

thanks