-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Max Rain Rate Does Not Reset #224
Comments
Indeed, the highlighted value does not reset. |
I changed the data feed to the MQTT from my weewx instance that uses a console logger. I'm attaching its output if you need to see it for some reason. |
Ok, I think I found the bug. Attached is an updated live-updates.js file, please feel free to test this one out (it should alert "rainRate" on first load). Take your time! I got the luck to have rain for the last 3 days to test this out in real-world conditions. |
Thanks, we're not expecting rain until the weekend. I do notice an odd behavior. |
To make the two value issue more clear, I have a short video of this. Let me know after you view it so I can remove the file. |
I downloaded the video, thank you. I will have a look at it as soon as possible, reporting back! |
I see what is happening, but I can not reproduce it. Realy can't make any sense out of it. Do you probably have a second MQTT-enabled skin installed to prove if it's the skin's fault or another problem? |
I shall install belchertown in the morning. It is installed on the live machine and uses the same MQTT feed as this. But to remove any difference, I'll install on the same machine. |
My main server has a rather odd configuration. |
The So if you are accessing the webpage:
See https://github.com/Daveiano/weewx-wdc/wiki/WeeWX-related#setting-html_root-and-basepath |
https://weather.ghammer.net/wdc/ [[WdcReport]] |
Sorry, it needs the ending slash:
|
Bingo! No issue on my production server. This is the release version. I'll add the 'rain' version in the morning. |
Well, I think I will wipe and reinstall the test server. I'll let it go through the rain that is forecast to be certain the rate is reset first. Taking lots of errors now. This is with the 'rain' file installed there. It is not on the production server and all is well there, no errors. |
So this sounds like the "alternating two value issue" is solved, right?
I have never seen such errors while developing. This is with weewx 4.10.2, right? |
On the test server/instance, the alternating values continue. It encompasses apptemp, heat index, wind chill, and dew point. I installed, with your help, on my production machine. It runs weewx 4.10.2. No errors in the logs and no alternating values. I plan to keep the test instance until rain comes to test the fix for the reset. Afterwards, I'm going to install Debian and weewx fresh and install the current release of WDC skin. |
Ok so the updated "rain file" should not produce these errors, it's not involved when the files are generated by Cheetah, so I assume this is WeeWX v5 related. I did not test the skin with v5 yet. How is the current progress, are we near a stable v5 release, yet?
Sounds great! |
V5 is stable, there are packaging issues as I understand. Rain forecast for my overnight, so Sunday morning I'll know if the reset functions. |
Well, I see that we have had rain since midnight. |
Max rain rate did reset with the 'rain' skin. |
The alternating values was apparently caused by the MQTTSubscribe driver. I returned to the WLL driver and the issue is gone. |
Great, then this change will come to the next release.
I think I know what's causing this, looks like a bug. I will give you an updated file for testing within this week, if you do not find any other bugs, this version will be in the next release. Thank you for all the testing! |
Sounds good. Thanks for your rapid fixes! |
Here is an updated file (this time without an alert) which should fix the problem described in #224 (comment) AND the rain rate issue. So please feel free to test this out extensively and report back any problems! |
Thanks. File is loaded on both systems. I'll report back afterwards. |
Rain yesterday, all was well. As it is our Thanksgiving holiday, I won't be around until Monday. I'm happy with the results, so feel free to close this ticket. |
Great! Please feel free to open a new issue/discussion if you find another bug. I wish you a happy Thanksgiving! |
The max rain rate does not reset. The min may not but as it normally zero hard to tell.
That wasn't noticeable until now.
I have verified that all other min/max seem to be resetting.
Originally posted by @gary-hammer in #198 (comment)
The text was updated successfully, but these errors were encountered: