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

GF radar reflectivity for RRFS realtime runs #699

Closed
wants to merge 4 commits into from

Conversation

haiqinli
Copy link
Contributor

Description

(Instructions: this, and all subsequent sections of text should be removed and filled in as appropriate.)
Provide a detailed description of what this PR does.
This PR is for RRFS_A and RRFS_B realtime runs.

What bug does it fix, or what feature does it add?
It includes the convective precipitation unit bug fix in GF radar reflectivity, soil moisture bug fix and updates for dust modules, and also the C3 convection updates.

Is a change of answers expected from this PR? Yes.

Issue(s) addressed

Link the issues to be closed with this PR, whether in this repository, or in another repository.
(Remember, issues should always be created before starting work on a PR branch!)

  • fixes #<issue_number>
  • fixes noaa-emc/fv3atm/issues/<issue_number>

Testing

How were these changes tested? It was tested with regression tests.

What compilers / HPCs was it tested with?
Intel and GNU compilers on Hera.

Are the changes covered by regression tests? (If not, why? Do new tests need to be added?) Yes

Have the ufs-weather-model regression test been run? On what platform?
Yes, on Hera.

  • Will the code updates change regression test baseline? If yes, why? Please show the baseline directory below.
    Yes, the regression test runs with GF, C3 and dust are changed due to the related code changes.

  • Please commit the regression test log files in your ufs-weather-model branch
    workflow.log

Dependencies

If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)

Do PRs in upstream repositories need to be merged first?
If so add the "waiting for other repos" label and list the upstream PRs

  • waiting on noaa-emc/nems/pull/<pr_number>
  • waiting on noaa-emc/fv3atm/pull/<pr_number>

SamuelTrahanNOAA added a commit to SamuelTrahanNOAA/fv3atm that referenced this pull request Sep 29, 2023
jkbk2004 pushed a commit that referenced this pull request Oct 3, 2023
…sics, and string length mismatch in dycore plus PR #705 and #699 (#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 <[email protected]>
Co-authored-by: Haiqin.Li <[email protected]>
@DusanJovic-NOAA
Copy link
Collaborator

Merged via #695

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