-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Decagon 5TM returns data inconsistently #135
Comments
@fisherba - This is really hard for me to debug seeing nothing other than some finally results that went to the website. I don't actually have a 5TM sensor available right now. It's possible that the 5TM isn't responding fast enough, that you have an older 5TM that isn't truly compatible with concurrent measurements (#115), or that something completely different is going on. I pushed a tweak to the 5TMspottiness branch increasing the allowed response time and being more careful to set values to -9999, but if you really want me to be able to help much more, I need you to post the debugging logs so I can look at what might be going wrong. To do that, checkout the new 5TM branch. Then open SensorBase.h and find and uncomment the line |
@fisherba - also make sure that all of your libraries are up-to-date. To do that in Atom/PlatformIO open up a command terminal (the button with the ">_" next to the serial monitor button). In the terminal type "pio lib update" and it should update the local version of all of your libraries. You can also run "pio lib -g update" if you've had PlatformIO install any libraries in another global directory instead of the .piolibdeps folder for your project. |
Also, if you're for any reason running NeoSWSerial or any of the "tone" functions, they will conflict with the SDI12 library. (You won't see any compile errors, just unexpected results.) |
Hi @SRGDamia1, sorry for the delay, this is the first I've been able to run the test. After running the test a few times, it was only running one test, so I let it run for about a half hour (while preparing a kid breakfast and lunch) and it didn't run any additional rounds of testing. Here's the output:
|
Uploading tonight for deployment tomorrow and on the first of my 5TMs when I used the EnviroDIY_ModularSensors@>=0.9.0, every other iteration of sensor test sent good values for 5TM. In the spottiness branch, every iteration of sensor test seems to send good values. What you've done is an improvement. We'll see how it does in the wild! |
Okay. I'm glad it seems to work. In the log you posted above it looks like it wasn't working. |
So, is this fixed? |
It's way better than the initial attempt. http://data.envirodiy.org/sites/NLP01/ |
Does that mean fixed.. or do you still want me to try and work on this? If it's fixed, please close the issue. |
According to @fisherba, the data from this summer gave a lot of bad values. @SRGDamia1, is the fix from this commit d347d50 likely to improve stability? |
I have no idea. |
I guess my question should have been rephrased: What was the motivation for that fix/commit? |
That "fix" to the 5TM was just because a few commits earlier that day (a72fe9b) I'd accidentally over-written the 5TM values with the D-Opto values when I was adding a few other notes. So it was just a fix to something I'd broken the same day and shouldn't make any difference at all to the actual working of the 5TM. |
Thanks. That's helpful to know. |
@fisherba reports this for site NLP01.
The text was updated successfully, but these errors were encountered: