-
Notifications
You must be signed in to change notification settings - Fork 172
update era5/ifs soil field processing #266
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
base: develop
Are you sure you want to change the base?
Conversation
@jimbresch @JoshLovesFun I downloaded the modified rd_grib2.F file, recompiled my WPS, and tested it with the operational ECMWF IFS 0.25º data using your Vtable.ERA5 and it didn't work. The soil temperature and moisture data still aren't being read. Maybe your contributions work for the ERA5 data, but they don't work for the operational IFS data. |
@gabrielcassol Thanks for your testing. I don't expect the Vtable.ERA5 to work for the operational ECMWF. It is designed to only work for the ERA5 data which I don't think had any changes to soil processing. Thus, a new Vtable specific to the latest operational ECMWF would need to be developed. Josh |
The fix @jimbresch provided is based on a particular dataset a user pointed to us. Depending on how your data is obtained, the data may have peculiarities. Can you point us to or supply us a data file that you have problem with? |
I will download and process IFS data to make sure what is going on. |
@jimbresch Do you think this is ready for WPS v4.7? |
@weiwangncar This bugfix was not run correctly when real.exe stoped with (grid%tsk unreasonable). |
Modify Vtable in line 20 to: WRF4.7 works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct line 20 to the follow:
235 | 1 | 0 | | SKINTEMP | K | Skin Temperature | 0 | 0 | 17 | 1 |
The WRF runs successfully.
@jimbresch I also tested it and can confirm it works with the modification by @andrewsoong I have two small suggestions though:
|
Modifications to rd_grib2.F to allow decoding of the era5/ifs soil fields. (referenced in #264 and #265). . Note that EC uses a local table for the grib codes and that g2print is not able to process these fields yet. Decoding was successful for a sample file.