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

Verification updates to METplus conf files from the RRFS_baseline branch #662

Merged

Conversation

JeffBeck-NOAA
Copy link
Collaborator

DESCRIPTION OF CHANGES:

Verification changes related to work from the RRFS_baseline branch are being brought over to the develop branch.

TESTS CONDUCTED:

End-to-end tests run in the context of the RRFS physics UFS-R2O project were conducted with these changes and confirmed to work correctly.

CONTRIBUTORS (optional):

@michelleharrold @jwolff-ncar

@jwolff-ncar
Copy link
Contributor

jwolff-ncar commented Jan 10, 2022

I want to add this to the conversation here to keep track of it and see if there are any further comments. For the RRFS_baseline test we had:
FCST_VAR10_NAME = HPBL
FCST_VAR10_LEVELS = Z0
OBS_VAR10_NAME = PBL
OBS_VAR10_LEVELS = L0
OBS_VAR10_OPTIONS = desc = "TKE"; censor_thresh = [ >=5000 ]; censor_val = [ -9999 ];

FCST_VAR11_NAME = HGT
FCST_VAR11_LEVELS = L0
FCST_VAR11_OPTIONS = GRIB_lvl_typ = 220;
OBS_VAR11_NAME = PBL
OBS_VAR11_LEVELS = L0
OBS_VAR11_OPTIONS = desc = "RI"; censor_thresh = [ >=5000 ]; censor_val = [ -9999 ];

There were changes to the MET code (see: dtcenter/MET#1913) to remove the need for this. However it has not been released yet (will be included in METv10.1.0). This is just a note that if you use METv10.0 you may want to add this censor_thresh/val setting back in.

Copy link
Contributor

@jwolff-ncar jwolff-ncar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I will also let @michelleharrold take a quick peak.

@JeffBeck-NOAA
Copy link
Collaborator Author

Looks good to me. I will also let @michelleharrold take a quick peak.

@jwolff-ncar, thanks for adding the note about PBL height!

Copy link
Collaborator

@michelleharrold michelleharrold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just gave it a look over, and everything looks good to me!

@JeffBeck-NOAA JeffBeck-NOAA merged commit 34ab11f into ufs-community:develop Jan 10, 2022
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

Successfully merging this pull request may close these issues.

3 participants