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

Correct the reading of inst. clear sky downward shortwave flux at surface for RRFS #956

Closed
wants to merge 2 commits into from

Conversation

EricJames-NOAA
Copy link
Contributor

This PR enables output of the instantaneous clear-sky downward shortwave flux at the surface for RRFS, which is required since this field was output in NAM-Nest.

The code was tested for a RRFS_CONUS_3km retrospective run on Jet.

@WenMeng-NOAA
Copy link
Collaborator

@EricJames-NOAA Can you provide the sample model files for my testing?

@EricJames-NOAA
Copy link
Contributor Author

@EricJames-NOAA Can you provide the sample model files for my testing?

@WenMeng-NOAA I am copying test files here on Hera (a 1-h forecast from 2023072903):

/scratch1/BMC/wrfruc/ejames/wen_clearsky/

The scp is quite slow for some reason...

@EricJames-NOAA
Copy link
Contributor Author

@EricJames-NOAA Can you provide the sample model files for my testing?

@WenMeng-NOAA I am copying test files here on Hera (a 1-h forecast from 2023072903):

/scratch1/BMC/wrfruc/ejames/wen_clearsky/

The scp is quite slow for some reason...

The files are copied now!

@WenMeng-NOAA
Copy link
Collaborator

@EricJames-NOAA Since the changes in this PR combined in #929, can you close it?

WenMeng-NOAA pushed a commit that referenced this pull request May 10, 2024
…RRFS (#929)

* Updates to use GRIB2 template 4.9 for QPFFFG and QPFARI variables in RRFS.

* Updating modulefiles for latest version of g2tmpl.

* Updating s4.lua for latest g2tmpl.lua

* Updating all postxconfig files based on the new required format (change to POST-XML-Library-NT.pl)

* Updates associated with PR #956 to speed up testing.

* Removing unneeded loading of g2tmpl on Jet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants