ufs-community / ufs-weather-model

UFS Weather Model
Other
129 stars 238 forks source link

[production/RRFS.v1] Fix improperly assigned fire emissions for ebb_dcycle==1 for retrospectives (NOT operational!) #2214

Closed jordanschnell closed 1 month ago

jordanschnell commented 1 month ago

Commit Queue Requirements:

Commit Message:

Fix improperly assigned fire emissions for ebb_dcycle==1 for retrospectives (NOT operational!)

* UFSWM - 
  * AQM - 
  * CDEPS - 
  * CICE - 
  * CMEPS - 
  * CMakeModules - 
  * FV3 - 
    * ccpp-physics - Fix improperly assigned fire emissions for ebb_dcycle==1 for retrospectives (NOT operational!)
    * atmos_cubed_sphere - 
  * GOCART - 
  * HYCOM - 
  * MOM6 - 
  * NOAHMP - 
  * WW3 - 
  * stochastic_physics - 

Priority:

Git Tracking

Sub component Pull Requests:

UFSWM Blocking Dependencies:


Changes

Regression Test Changes (Please commit test_changes.list):

053 conus13km_control_intel failed in check_result conus13km_control_intel 053 failed in run_test 059 conus13km_debug_intel failed in check_result conus13km_debug_intel 059 failed in run_test 060 conus13km_debug_qr_intel failed in check_result conus13km_debug_qr_intel 060 failed in run_test 061 conus13km_debug_2threads_intel failed in check_result conus13km_debug_2threads_intel 061 failed in run_test 062 conus13km_radar_tten_debug_intel failed in check_result conus13km_radar_tten_debug_intel 062 failed in run_test

Input data Changes:

Library Changes/Upgrades:


Testing Log:

jkbk2004 commented 1 month ago

@jordanschnell @MatthewPyle-NOAA Should we prepare to schedule to commit this pr to the rrfs.v1 production branch? Should same feature be added to develop branch?

grantfirl commented 1 month ago

@jordanschnell Please pull in the latest production/RRFS.v1 code for testing.

MatthewPyle-NOAA commented 1 month ago

@jordanschnell @MatthewPyle-NOAA Should we prepare to schedule to commit this pr to the rrfs.v1 production branch? Should same feature be added to develop branch?

@jkbk2004 yes on scheduling to get this into the rrfs.v1 production branch. I don't know enough about it to know if it also needs to go to the develop branch.

jkbk2004 commented 1 month ago

@MatthewPyle-NOAA I am going to start testing on hera and hercules. @grantfirl we can figure out how to bring another pr to develop branch, right?

MatthewPyle-NOAA commented 1 month ago

@jkbk2004 I've started testing on WCOSS.

MatthewPyle-NOAA commented 1 month ago

My initial test on WCOSS confirmed the @jordanschnell result of those five tests failing.

grantfirl commented 1 month ago

@MatthewPyle-NOAA I am going to start testing on hera and hercules. @grantfirl we can figure out how to bring another pr to develop branch, right?

@jkbk2004 Yes. I can add it with the others. I need to figure out whether some of the PRs into the RRFS production branch can be safely combined into fewer PRs for develop.

jkbk2004 commented 1 month ago

@MatthewPyle-NOAA note that I updated bl_date.conf to BL_DATE=20240410 since 5 cases changes results. Can you make sure about that on wcoss2?

MatthewPyle-NOAA commented 1 month ago

@jkbk2004 - yes, I did that on WCOSS2. But you've reminded me that I need to mirror the new develop-20240410 directory to the other side of WCOSS2.

jkbk2004 commented 1 month ago

@jkbk2004 - yes, I did that on WCOSS2. But you've reminded me that I need to mirror the new develop-20240410 directory to the other side of WCOSS2.

Then, we can start merging process.

jordanschnell commented 1 month ago

@grantfirl , tried the same procedure -- met with:

Screenshot 2024-04-11 at 1 34 22 PM

jkbk2004 commented 1 month ago

@grantfirl , tried the same procedure -- met with:

Screenshot 2024-04-11 at 1 34 22 PM

Sounds like I raced to push. But this pr looks good to merge.

jordanschnell commented 1 month ago

@grantfirl , tried the same procedure -- met with: Screenshot 2024-04-11 at 1 34 22 PM

Sounds like I raced to push. But this pr looks good to merge.

Thanks, @jkbk2004!