ufs-community / ufs-weather-model

UFS Weather Model
Other
129 stars 238 forks source link

[production/RRFS.v1] Update module_write_netcdf to avoid hangs in RRFS runs #2212

Closed DusanJovic-NOAA closed 1 month ago

DusanJovic-NOAA commented 1 month ago

Commit Queue Requirements:

Commit Message:

* UFSWM - 
  * AQM - 
  * CDEPS - 
  * CICE - 
  * CMEPS - 
  * CMakeModules - 
  * FV3 - 
    * ccpp-physics - 
    * atmos_cubed_sphere - 
  * GOCART - 
  * HYCOM - 
  * MOM6 - 
  * NOAHMP - 
  * WW3 - 
  * stochastic_physics - 

Priority:

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:


Changes

Regression Test Changes (Please commit test_changes.list):

Input data Changes:

Library Changes/Upgrades:


Testing Log:

hu5970 commented 1 month ago

The tests with RRFS_A on development machine are successful. Please merge this PR.

jkbk2004 commented 1 month ago

@MatthewPyle-NOAA #2193 is on-going. So it's good timing to work on this pr as well. I need to test if we can use Hera rocky8 in this pr.

jkbk2004 commented 1 month ago

@DusanJovic-NOAA can you convert draft pr to pr?

jkbk2004 commented 1 month ago

@MatthewPyle-NOAA We can start merging process if we don't need to test on more than hera/hercules/wcoss2 machines. #2193 ran all ok. So, I think we are good to merge the pr.

MatthewPyle-NOAA commented 1 month ago

@jkbk2004 Can help with merging later today - let me know what you need me to do.

jkbk2004 commented 1 month ago

@DusanJovic-NOAA can you update fv3 hash: https://github.com/NOAA-EMC/fv3atm/commit/c01786de120534604f686e206c5be6005d96f55b ?

DusanJovic-NOAA commented 1 month ago

@DusanJovic-NOAA can you update fv3 hash: NOAA-EMC/fv3atm@c01786d ?

Done.

jkbk2004 commented 1 month ago

@MatthewPyle-NOAA All looks good. This pr can be merged.