Skip to content

Enabling GSL p-type, winter QPF and buckets, and lightning threat indices for RRFS#695

Merged
WenMeng-NOAA merged 6 commits into
NOAA-EMC:developfrom
EricJames-NOAA:winter_qpf
Apr 25, 2023
Merged

Enabling GSL p-type, winter QPF and buckets, and lightning threat indices for RRFS#695
WenMeng-NOAA merged 6 commits into
NOAA-EMC:developfrom
EricJames-NOAA:winter_qpf

Conversation

@EricJames-NOAA
Copy link
Copy Markdown
Contributor

This PR switches to the GSL precip type algorithm for RRFS, introduces new winter QPF output (TSNOWP, FRZR, FROZR, both run total accumulated and 1-h accumulated), and enables output of the three GSL lightning threat diagnostics.

The changes were tested retrospectively for the RRFS-B (3km CONUS) system on Jet.

@EricJames-NOAA EricJames-NOAA changed the title Enabling GSL p-type, winter QPF and buckets, and lightning threat indices Enabling GSL p-type, winter QPF and buckets, and lightning threat indices for RRFS Apr 21, 2023
@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

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

@EricJames-NOAA
Copy link
Copy Markdown
Contributor Author

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

I copied some test files here on Hera:

/scratch1/BMC/wrfruc/ejames/wen_ptype/

@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

@EricJames-NOAA Thanks!

@WenMeng-NOAA WenMeng-NOAA linked an issue Apr 21, 2023 that may be closed by this pull request
Comment thread parm/post_avblflds.xml Outdated
Comment thread parm/fv3lam_rrfs.xml Outdated
@ericaligo-NOAA
Copy link
Copy Markdown
Contributor

ericaligo-NOAA commented Apr 22, 2023 via email

Comment thread parm/post_avblflds.xml
Copy link
Copy Markdown
Contributor

@MatthewPyle-NOAA MatthewPyle-NOAA left a comment

Choose a reason for hiding this comment

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

Generally looked good to me - just one question about the removal of the "MAX" for the two lightning products, and a cosmetic suggestion for code readability.

Comment thread sorc/ncep_post.fd/SURFCE.f
Copy link
Copy Markdown
Contributor

@ericaligo-NOAA ericaligo-NOAA 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.

Comment thread parm/post_avblflds.xml
Comment thread parm/post_avblflds.xml
@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

@EricJames-NOAA My test results are at /scratch1/NCEPDEV/stmp2/Wen.Meng/rrfs_2023042020 on Hera. I see the following expected changes:
New variables added in NATLEV dataset:
1309:1118304488:d=2023042020:LTNGSD:1 m above ground:2-3 hour max fcst:
1310:1118319328:d=2023042020:LTNGSD:2 m above ground:2-3 hour max fcst:

values change in CRAIN, CSNOW, CICEP, CFRZR due to switching to GSL P-types.

Please let me know if you see any issues. Thanks!

@EricJames-NOAA
Copy link
Copy Markdown
Contributor Author

@EricJames-NOAA My test results are at /scratch1/NCEPDEV/stmp2/Wen.Meng/rrfs_2023042020 on Hera. I see the following expected changes: New variables added in NATLEV dataset: 1309:1118304488:d=2023042020:LTNGSD:1 m above ground:2-3 hour max fcst: 1310:1118319328:d=2023042020:LTNGSD:2 m above ground:2-3 hour max fcst:

values change in CRAIN, CSNOW, CICEP, CFRZR due to switching to GSL P-types.

Please let me know if you see any issues. Thanks!

@WenMeng-NOAA thanks for your test! The results look good to me!

Copy link
Copy Markdown
Contributor

@MatthewPyle-NOAA MatthewPyle-NOAA left a comment

Choose a reason for hiding this comment

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

Thanks @EricJames-NOAA for addressing my comments and questions. Looks good to me now.

@WenMeng-NOAA WenMeng-NOAA added Baseline Change The baselines of the UPP regression tests are changed. Ready for Review This PR is ready for code review. labels Apr 24, 2023
@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

@EdwardColon-NOAA Please be aware of the changes which have impact on 3DRTMA in this PR.

@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

The UPP RT tests were completed on WCOSS2, Hera and Orion. The baselines for 3DRTMA, RAP and HRRR will be recreated with this PR.

@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

@ericaligo-NOAA @MatthewPyle-NOAA Thanks for reviewing this PR.

@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

This PR is ready for merging.

@EdwardColon-NOAA
Copy link
Copy Markdown
Contributor

EdwardColon-NOAA commented Apr 25, 2023 via email

@WenMeng-NOAA
Copy link
Copy Markdown
Collaborator

Hi Wen, These changes shouldn't impact the use of 3D-RTMA. Thanks, Edward

On Mon, Apr 24, 2023 at 7:59 PM WenMeng-NOAA @.> wrote: @ericaligo-NOAA https://github.com/ericaligo-NOAA @MatthewPyle-NOAA https://github.com/MatthewPyle-NOAA Thanks for reviewing this PR. — Reply to this email directly, view it on GitHub <#695 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACFXXQAIKMC3FICRTJXOYN3XC4HULANCNFSM6AAAAAAXHBQS6I . You are receiving this because you were mentioned.Message ID: @.>
-- ------------------------------------------------- Edward Colón Lynker at NOAA/NWS/NCEP/EMC 5830 University Research Ct. Rm #2025 Riverdale, MD 20737 Office: (301) 683-3815 Cell: (301) 213-3566

@EdwardColon-NOAA Thanks for confirming.

@WenMeng-NOAA WenMeng-NOAA merged commit 7e58735 into NOAA-EMC:develop Apr 25, 2023
This was referenced Jun 21, 2023
Comment thread sorc/ncep_post.fd/INITPOST_NETCDF.f
zhanglikate pushed a commit to zhanglikate/UPP that referenced this pull request Oct 10, 2025
…sics, and string length mismatch in dycore plus PR NOAA-EMC#705 and NOAA-EMC#699 (NOAA-EMC#695)

* GFDL_atmos_cubed_sphere: consistent string lengths in array

* stop FV3_HRRR_c3 from crashing with gnu debug

* in dycore, initialize srf_wnd_var2 and tracers_var3 arrays

* Write netcdf axis variables using the same real kind as data variables

* "GF radar reflectivity, dust bug fix, C3 updates, more fluxes output"

* Fix race condition in GFS_phys_time_vary.fv3.F90 error detection

* More bug fixes to GFS_phys_time_vary.fv3.F90:
1. detect empty errmsg from subroutines
2. Initialize err variables in set_soilveg.f, which is called from GFS_phys_time_vary.fv3.F90

* initialize errmsg & errflg in noahmp_tables.f90

* only read h2odata, ozdata and noahmp table when they are needed

* remove all constant 3D variables from clm lake

* calculate clm lake constants only once per i loop

---------

Co-authored-by: Dusan Jovic <dusan.jovic@noaa.gov>
Co-authored-by: Haiqin.Li <Haiqin.Li@noaa.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3DRTMA Baseline Change The baselines of the UPP regression tests are changed. Ready for Review This PR is ready for code review. RRFS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable GSL precip type algorithm for RRFS

6 participants