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

Add accumulated snow melt and surface albedo output in RRFS #904

Closed
EricJames-NOAA opened this issue Mar 25, 2024 · 3 comments · Fixed by #908
Closed

Add accumulated snow melt and surface albedo output in RRFS #904

EricJames-NOAA opened this issue Mar 25, 2024 · 3 comments · Fixed by #908
Assignees
Labels

Comments

@EricJames-NOAA
Copy link
Contributor

Accumulated snow melt (SNOM) and surface albedo are needed in RRFS. Also, we need to encode the total soil moisture as CISOILM.

@WenMeng-NOAA
Copy link
Collaborator

@EricJames-NOAA The UPP currently reads total soil moisture "soilm" from UFS based model output and writes it out as grib2 with name "SOILMOI". Could you clarify your intention of changing grib2 name from "SOILMOI" to "CISOILM"?

@EricJames-NOAA
Copy link
Contributor Author

@WenMeng-NOAA yes, the previous code was an error. The soilm variable from UFS is in kg/m2, so it should be encoded as CISOILM (also in kg/m2). This code change is a bug fix.

@WenMeng-NOAA
Copy link
Collaborator

@WenMeng-NOAA yes, the previous code was an error. The soilm variable from UFS is in kg/m2, so it should be encoded as CISOILM (also in kg/m2). This code change is a bug fix.

@EricJames-NOAA I see. Thanks for catching that bug. I will change GFS products later.

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

Successfully merging a pull request may close this issue.

2 participants