RRFSv1 updates: new fields#1473
Conversation
…threshold concentration
|
@EricJames-NOAA Thanks for submitting this PR to add these new fields for RRFSv1. I'll review and test your changes today. @WenMeng-NOAA Let's process this PR next. We would like to get these new fields into the RRFS real-time parallel soon. |
|
@EricJames-NOAA I tested your changes on Ursa and I see 6 new fields in the 2DFLD GRIB2 file:
Here is my run directory on Ursa if you could check and make sure the results look good: |
sorc/ncep_post.fd/CALPW.f
Outdated
| INTEGER LLMH,I,J,L | ||
| REAL ALPM,DZ,PM,PWSUM,RHOAIR,DP,ES | ||
| REAL QDUM(ista:iend,jsta:jend), PWS(ista:iend,jsta:jend),QS(ista:iend,jsta:jend) | ||
| REAL COLMD(ista:iend,jsta:jend), COLMD2(ista:iend,jsta:jend) |
There was a problem hiding this comment.
@EricJames-NOAA May I suggest using dynamic arrays (allocatable) for these two new arrays and deallocating them at the end of this subroutine?
@BenjaminBlake-NOAA thanks for the tests. I would like to double check the dust emission field, since the netCDF field was all zero in my earlier test. Can you share the dyn and phy files for your test run? Thanks! |
|
@EricJames-NOAA Sure the dyn/phy files are located here: |
@EricJames-NOAA The RRFS model output @BenjaminBlake-NOAA provided are from our UPP RTs. It appears to me a new variables will be read from RRFS model output in this PR. Would you provide us your new sample RRFS output for the RRFS standalone test? |
@WenMeng-NOAA the new variable read in (emdust) is already available in these files...we just haven't passed it into UPP before. I checked Ben's test and made one additional changes (correcting units of dust emissions). Based on my tests, the new code appears to be working as intended. |
Co-authored-by: WenMeng-NOAA <48260754+WenMeng-NOAA@users.noreply.github.com>
Co-authored-by: WenMeng-NOAA <48260754+WenMeng-NOAA@users.noreply.github.com>
|
@GangZhao-NOAA @jaymes-kenyon Please note that the correct grib2 name for fire radiative power is implemented in this PR, which impacts on 3DRTMA and MPAS products. |
|
This PR looks good! Once merged, I anticipate that several merge conflicts will arise within PR #1455, but I will fix those conflicts. |
|
Thanks for @BenjaminBlake-NOAA conducting the UPP RTs on WCOSS2. |
|
The UPP RTs have been completed on WCOSS2 and all expected changes have been confirmed. There will be baseline updates in the rrfs, mpas, 3drtma, rap, hrrr tests. |
|
@clyden-noaa This PR is ready for testing on the RDHPCs. As @WenMeng-NOAA noted, baseline changes are expected for the rrfs, mpas, mpas_hfip, 3drtma, rap, and hrrr tests. I generated baseline data for you here on Ursa: Thanks! |
GangZhao-NOAA
left a comment
There was a problem hiding this comment.
The modifications look good to me.
Thanks!
|
@clyden-noaa We'd like to update the input data for the RRFS baseline test with this PR. The current input files are about a year old. Please hold off on your testing for now and I'll let you know once everything is ready. |
|
@clyden-noaa We can go ahead and proceed with testing. I have new RRFS input files for you here on Ursa: The baseline output data are in the same locations as before: Baseline changes are expected for the rrfs, mpas, mpas_hfip, 3drtma, rap, hrrr, and rrfs_ifi_missing tests. rrfs_ifi_missing is now changed because we updated the RRFS input files. Once Orion and Hercules are back online I can copy the RRFS input files over to those systems for you. Thanks! |
|
@BenjaminBlake-NOAA Thanks for updates in rrfs test with new data_in baseline and related job cards. I also confirm that the proposed new rrfs variables are in place. |
clyden-noaa
left a comment
There was a problem hiding this comment.
@WenMeng-NOAA @BenjaminBlake-NOAA @EricJames-NOAA
My testing has been completed
There are baseline changes for the following:
- mpas
- POSTTWO18.tm00
- hrrr
- WRFTWO.GrbF10
- mpas_hfip
- 2DFLD.GrbF48
- rrfs_ifi_missing
- IFIFIP36.tm00
- 3drtma
- WRFTWO.GrbF00
- rrfs
- NATLEV36.tm00
- 2DFLD36.tm00
- PRSLEV36.tm00
- rap
- WRFPRS.GrbF06
- WRFNAT.GrbF06
There were new input files for rrfs:
- dynf036.nc
- phyf036.nc
All input files were copied to their resepective test suites on all RDHPCs
All baseline files were created and copied to their proper locations on all RDHPCs for both intel (ursa, hercules, orion) and intelllvm (ursa) compilers
Following baseline file creation, all RTs ran successfully and within established timing thresholds on all RDHPC systems.
Ursa - intel
Check tests:
['sfs', 'aqm', 'gefsv12', 'gefsv13', 'nmmb', 'rap', 'hrrr', 'hafs', '3drtma', 'mpas', 'mpas_hfip', 'gcafs', 'rrfs', 'rrfs_ifi_missing', 'gfs']
No changes in test results detected.
Ursa - intelllvm
Check tests:
['sfs', 'aqm', 'gefsv12', 'gefsv13', 'nmmb', 'rap', 'hrrr', 'hafs', '3drtma', 'mpas', 'mpas_hfip', 'gcafs', 'rrfs', 'rrfs_ifi_missing', 'gfs']
No changes in test results detected.
Orion - intel
Check tests:
['sfs', 'aqm', 'gefsv12', 'gefsv13', 'nmmb', 'rap', 'hrrr', 'hafs', '3drtma', 'mpas', 'mpas_hfip', 'gcafs', 'rrfs', 'rrfs_ifi_missing', 'gfs']
No changes in test results detected.
Hercules - intel
Check tests:
['sfs', 'aqm', 'gefsv12', 'gefsv13', 'nmmb', 'rap', 'hrrr', 'hafs', '3drtma', 'mpas', 'mpas_hfip', 'gcafs', 'rrfs', 'rrfs_ifi_missing', 'gfs']
No changes in test results detected.
BenjaminBlake-NOAA
left a comment
There was a problem hiding this comment.
@clyden-noaa Thanks for testing and updating the baselines.
@EricJames-NOAA Thanks for adding these new products to RRFS. I will work on getting them added to the real-time parallel soon.
@WenMeng-NOAA Unless you have any final concerns, this PR is ready for merging.
This PR adds several new fields for RRFSv1, based on developer and user requests:
(1) Vertically integrated total dust (COLMD), which is a sum of COLMD for coarse dust and fine dust
(2) Fire radiative power (FRADPOW)
(3) Total dust emissions (AEMFLX)
(4) Aerosol layer top (HGTMD), which is highest height where smoke + total dust exceeds 1 ug/m3
(5) Aerosol layer bottom (HGTMD), which is lowest height where smoke + total dust exceeds 1 ug/m3
(6) Aerosol layer mass-weighted centroid (HGTMD)
The code was tested on Jet for RRFSv1 output.