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

WGR800 error - wind speeds >10m/s #9

Open
jrhilton opened this issue Aug 28, 2020 · 7 comments
Open

WGR800 error - wind speeds >10m/s #9

jrhilton opened this issue Aug 28, 2020 · 7 comments

Comments

@jrhilton
Copy link

jrhilton commented Aug 28, 2020

Hi, I've noted that wind speed never goes above 36km/h (10m/s) yet when I look on the OS consol during high winds it does.

The speeds match up exactly to the consol otherwise, except during very high winds. What could be causing this issue?

Almost as if it should be something like this to also get the tens, as I can see it gets the ones, and decimal tenths:
double gust = ((nyb(14) * 100) + (nyb(13) * 10) + nyb(12)) * 3.6 / 10;

and
avWindspeed = ((nyb(17) * 100) + (nyb(16) * 10) + nyb(15)) * 3.6 / 10;

Not had a chance to upload this to try, but has anyone else noticed this issue?

Many thanks,
James.

P.s great work on this, I can't tell you how useful it has been, as getting data out of the console via USB is hit and miss!!!

@robwlakes
Copy link
Owner

robwlakes commented Aug 29, 2020 via email

@jrhilton
Copy link
Author

jrhilton commented Aug 29, 2020

Hi Rob, many thanks for your reply. I've had another read of this document that details the RF Protocols and it does spec three nibbles making up the speeds, so adds some substance to my logic. The numbering of the nibbles is slightly offset in that document vs your code, but if you normalise them using the wind direction nibbles then there is a perfect overlap when you add in nyb(14) and nyb(17) to your code.

I've now added this code:

gustActualMS_tst = ((nyb(14) * 100) +(nyb(13) * 10) + nyb(12)); //new test
avWindspeedMS_tst = ((nyb(17) * 100) + (nyb(16) * 10) + nyb(15));//new test

Which are also sent via the serial port using dumpAnemom() and logged. I may need someone else to help validate this too as wind speeds >10m/s are rare here in London, UK. So it maybe many months before I get some actual data to validate it myself, but I will be sure to report back when I do have some data.

Stay safe and well in these times!

@robwlakes
Copy link
Owner

robwlakes commented Aug 30, 2020 via email

@jrhilton
Copy link
Author

jrhilton commented Sep 6, 2020

Thanks Rob, as an update still no high winds here, plenty of sunshine instead so we shouldn't complain!

So not got a chance to test the changes at this end yet. Though as a side note I have ported the code over to run on an ESP8266 which interestingly when using the same type of receiver module and stock antenna is far more sensitive and picks up all transmissions even on the other side of the house where the Arduino plus the same receiver module would be deaf. And that is with the receiver running at 3.3v instead of 5v. Either i've got super lucky with a very sensitive receiver module when running at 3.3v or the ESP8266 has much better timings (which I suspect is the case).

Kind regards,
James

@robwlakes
Copy link
Owner

robwlakes commented Sep 6, 2020 via email

@roberto1984
Copy link

Hi all guys, hi Rob!
i'm also using the esp8266 since 1 year ,starting from rob code that i have adapted for esp8266 nodemcu, uploading every 50 seconds to a mysql database.
I also confirm that the wind was "capped" at 10m/s, just today i added the fix from jrhuilton(thank you!!!!),i will let you know if the reading between console and nodemcu are aligned!

Cheers!
Roberto

@jrhilton
Copy link
Author

jrhilton commented Dec 28, 2020

hello @robwlakes / @roberto1984 - I hope you are both keeping well and safe! Sorry for such a delayed update but this week we had the first winds >10m/s since my last post. As a result of Storm Bella I recorded a max of 59.76 km/h on 27th December from one transmission which the console didn't record (not surprising as I note it doesn't register every transmission anyway). However it shows a max of 49.6km/h which I have correlated to the data in my SQL database for the same time. So I am satisfied that my code suggestion works and is matching to the console.

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

No branches or pull requests

3 participants