-
Notifications
You must be signed in to change notification settings - Fork 1
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
Wind speed and wind direction issues R.06 #79
Comments
Another issue with the hourly_qff_to_cdm_obs_v1.py wind speed code section is not picking up the observed wind speed value or the original wind speed value from the df (code section ws starts line 490-530) . I do not think there is any conversion in wind speed values. Needs the piece of code to add the values see other variables. |
… exists in utils.construct_extra_ids()
OK - I think this now works. There is quite some confusion through the scripts (Obs and Lite checked so far) as to how NaN, null and Null values are being created, masked and dropped. This meant that some code lines aren't doing anything in the Currently this routine now does drop rows without observation values, and the fix for the masked wind speed and direction measurement codes also works. Issue #80 raised to address other items once Rel 6 completed. |
Thanks for doing these edits. |
TBH, I'm not sure. I presumed that it would be the same instrument so handled in the same way. But I don't actually know. |
* #79: updated singificance and duration in dataframe * #79: updated singificance and duration in dataframe for obs table * #79: added structures to manage flags and empty routine * #79: initial attempt at masking all but the retained codes * #79: corrected wind speed obs_value assignment. Replaced code already exists in utils.construct_extra_ids() * #79: fixing handling of empty data rows. Further improvements needed however * #79: added the startswith() matching * #79: updating list of wind measurement codes to retain
We have found that some new average wind observations over specific time frames have been introduce into the r.06 merge.
For example CAW00064757.qff has 60minute average wind speed obseravtions enocded in the measurement code column as “R-60min-av”. In the C3S release we are currently only using "N" normal and "C" calm hourly wind speed observations.
For example I checked the CAW00064757.qff from source (NCEI/CRN) for occurrences of “R-60min-av” in the wind_speed_Measurement_Code columns and the file has 148967 rows of wind observations at “R-60min-av” out of the total rows of 1743012 over the period (2005 -2021). There are the same amount of wind_direction rows with -999.0. There are no other wind observations in this files beside the “R-60min-av”.
The corresponding wind direction for the R-60min-av wind speed observation is set to -999.0 because it is difficult to calculate hourly average wind direction.
Apparantly there are other potential new averages of wind obseravation in the r.06 merge (see list below). We will also need to check if other variables have averages introduced.
Potential solutions
Remove codes for C3S
'A':'A-Abr-Beauf',
'B':'B-Beaufort',
'H':'H-5min-avg-spd',
'R':'R-60min-avg-spd',
'Q':'Q-Squall',
'T':'T-180min-avg-spd',
'V':'V-Variable',
'9':'9-Missing'
Keep codes for C3S:
'C':'C-Calm',
N':'N-Normal',
Team decision
The team agreed that it is a lot to get our heads around for r.06 to proceed with option 1 at this time. So we will only bring forward "N" normal and "C" calm wind observations in the C3S release but leave in them in the qff for NCEI to use in GHCNh. Lets look at the balance of wind observations over the next year and make an informed decision on whether/what we include them in r.07.
Other wind issue
It has also come to our attention that we will have to change the current encoding for wind speed observations as they are 10-min averges and not instatenous measurements.
At the moment all hourly wind speed observations in cdm format files are set to observed_value [106] = wind speed, duration [0] = instantaneous and significance value [12] = instantaneous.
The QFF to cdm_lite and cdm_obs CDM conversion code should be changed to:
At the moment all hourly wind speed observations in cdm format files are set to observed_value [106] = wind speed, duration [8] = 10 minutes and significance value [2] = mean.
The text was updated successfully, but these errors were encountered: