-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Implement Campbell RainVUE10 SDI-12 Precipitation Sensor #416
Comments
I have this mostly working, but for some reason, ModularSensors isn't properly parsing the Get Measurement response (or something like that): Here's what I get from Serial Monitor when running the simple_logging_rainvue.ino test sketch with
@SRGDamia1, any suggestions? I have much less experience with SDI-12 than I do with Modbus. |
I just tried also defining
|
Here's my output from /EnviroDIY/Arduino-SDI-12/examples/d_simple_logger, set to run SDI-12 from Mayfly pin 7 (switched 3.3V). It shows 3 tips at 51806 ms elapsed time and 4 tips at 106764 ms.
|
Similar to above (/EnviroDIY/Arduino-SDI-12/examples/d_simple_logger), but set to run SDI-12 from Mayfly pin 5 (constant 3.3V). Note that this immediately finds the sensor info, whereas it did not find when on switched power.
|
@SRGDamia1, I'm quite stumped by this. Any suggestions on what I might try? |
I've played around with Warm Up and Measurement times, from 0 to 10,000 ms, and tried a 30 ms Extra Wake time. I've tried concurrent and non-concurrent, and right now I'm playing around with modifying the None work. |
I now have some results, but not yet the ones I need. I changed the
Here's the serial monitor response.
|
@gcutrell, do you have any suggestions on what I might try? |
Does example k in the SDI-12 library work? |
@SRGDamia1, thanks for the tip! Example K does not work. Here is the serial monitor response:
|
@aufdenkampe @SRGDamia1 I think the issue is in SDI12Sensors.cpp, the program doesn't allow for a zero wait time. I recently discovered this and just added the code below before "if (wait > 0) {" in the concurrent and non-concurrent sections. |
@gcutrell, that did it! Thanks!!!
|
Thanks to suggestion from @gcutrell in #416, comment: #416 (comment)
Sorry, sloppy on my part. In the |
@SRGDamia1, thanks for a cleaner solution! One last interesting thing: all this only works when I define If I turn off debugging flags, or only define I'm taking off the rest of the week, so I won't be able to sleuth out the cause of this until mid-next week at the earliest. |
There was a bug in the non-concurrent measurement that was making it freeze without the debugging. I found it last week when Shannon needed to deploy some really old 5TM soil sensors. That should be fixed in one of the commits I pushed to your branch from master. |
@SRGDamia1, I forgot to follow up and let you know that it all worked, and the station was deployed a couple of days later. Data have been streaming nicely (with a few -9999 values here and there...). See https://monitormywatershed.org/sites/WOODB-Well18/ I'll issue a PR to |
Implement ModularSensors wrapper for Campbell RainVUE10 SDI-12 Precipitation Sensor with Plastic Funnel.
Using the following Campbell documentation
Implementing within the
RainVUE
branch.The text was updated successfully, but these errors were encountered: