Add a RT for SFS with hydrostatic options #2255

XiaqiongZhou-NOAA · 2024-04-26T13:58:17Z

Commit Queue Requirements:

Fill out all sections of this template.
All sub component pull requests have been reviewed by their code managers.
Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
Commit 'test_changes.list' from previous step

Description:

Add a RT for the seasonal forecast system with hydrostatic options.
Compiling flags -DUSE_COND and -DMOIST_CAPPA are removed if this is a hydrostatic run.
inline post is modified to output omega and HGT for hydrostatic runs.

Commit Message:

* UFSWM  - Add a RT for SFS with hydrostatic options
  * FV3 - Remove -DUSE_COND and -DMOIST_CAPPA compiling flags for hydrostatic runs;
              Fix the issue to write out geopotential height for inline post in hydrostatic runs; 
              Use omega from the model if defined in diag_table instead calculated in post
       * atmos_cubed_sphere - Remove -DUSE_COND and -DMOIST_CAPPA compiling flags for hydrostatic runs

Priority:

Normal.

Sub component Pull Requests:

FV3: Change compiling flag for hydrostatic options NOAA-EMC/fv3atm#828
GFDL_atmos_cubed_sphere Remove -DUSE_COND and -DMOIST_CAPPA for hydrostatic runs NOAA-GFDL/GFDL_atmos_cubed_sphere#337
Closes turn threaded tests back on for Derecho #2242

Changes

Regression Test Changes (Please commit test_changes.list):

PR add a new RT: cpld_gfsv17_sfs intel

Input data Changes:

None.

Library Changes/Upgrades:

No Updates

Testing Log:

zach1221 · 2024-05-10T18:06:00Z

@XiaqiongZhou-NOAA running ORTs against the new cpld_gfsv17_sfs case with gnu compilers. Right now getting some failures on the bit, debug, standard, decomp, and restart ORTs. The compile is fine, the failure occurs during the test run.
fyi @jkbk2004 @BrianCurtis-NOAA

"An MPI communication peer process has unexpectedly disconnected."

180: *** WAVEWATCH III ERROR IN W3IOGR :
180: ERROR IN READING FROM mod_def.ww3 FILE
180: IOSTAT = 5016 MOD DEF FILE WAS GENERATED WITH A DIFFERENT
180: WW3 VERSION OR USING A DIFFERENT SWITCH FILE.
180: MAKE SURE WW3_GRID IS COMPILED WITH SAME SWITCH
180: AS WW3_SHEL OR WW3_MULTI, RUN WW3_GRID AGAIN
180: AND THEN TRY AGAIN THE PROGRAM YOU JUST USED.

Example test experiment directory: /scratch1/NCEPDEV/stmp2/Zachary.Shrader/FV3_OPNREQ_TEST/opnReqTest_2606214/bit_base_bit_base/ looking at err and out logs.

BrianCurtis-NOAA · 2024-05-10T18:13:02Z

Check that the mod_def file from WW3 is the correct one

tests/rt.conf

JessicaMeixner-NOAA · 2024-05-10T18:40:22Z

@BrianCurtis-NOAA is correct. Based on rt.sh and your test, you should be using /scratch2/NAGAPE/epic/UFS-WM_RT/NEMSfv3gfs/input-data-20221101/WW3_input_data_20240214/mod_def.exp.global_270k for your mod_def, but:

Binary files /scratch1/NCEPDEV/stmp2/Zachary.Shrader/FV3_OPNREQ_TEST/opnReqTest_2606214/bit_base_bit_base/mod_def.ww3 and /scratch2/NAGAPE/epic/UFS-WM_RT/NEMSfv3gfs/input-data-20221101/WW3_input_data_20240214/mod_def.exp.global_270k differ

I think you're using the wrong WW3_input data somehow as: /scratch1/NCEPDEV/stmp2/Zachary.Shrader/FV3_OPNREQ_TEST/opnReqTest_2606214/bit_base_bit_base/mod_def.ww3 /scratch2/NAGAPE/epic/UFS-WM_RT/NEMSfv3gfs/input-data-20221101/WW3_input_data_20220624/mod_def.exp.global_270k
are the same.

NickSzapiro-NOAA · 2024-05-10T20:12:44Z

@zach1221 ORT INPUTDATA_ROOT_WW3 needs to match rt.sh:
https://github.com/ufs-community/ufs-weather-model/blob/develop/tests/opnReqTest#L636
https://github.com/ufs-community/ufs-weather-model/blob/develop/tests/rt.sh#L1037

zach1221 · 2024-05-11T16:05:41Z

ORTs passed with WW3_input_data_20240214.

zach1221 · 2024-05-13T17:55:24Z

@XiaqiongZhou-NOAA @NickSzapiro-NOAA Can you please sync up the branch for this PR?

XiaqiongZhou-NOAA · 2024-05-13T18:09:00Z

The branch is synced.

…

On Mon, May 13, 2024 at 1:55 PM zach1221 ***@***.***> wrote: @XiaqiongZhou-NOAA <https://github.com/XiaqiongZhou-NOAA> @NickSzapiro-NOAA <https://github.com/NickSzapiro-NOAA> Can you please sync up the branch for this PR? — Reply to this email directly, view it on GitHub <#2255 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALQE7UV5MGNEKEKFYEHHMCLZCD5CFAVCNFSM6AAAAABG22ZRWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBYGQ2DGMJZHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

BrianCurtis-NOAA · 2024-05-13T18:11:09Z

Looks like FV3ATM is supposed to be at: XiaqiongZhou-NOAA/fv3atm@fc2630c

This hash was after the indentation change I requested.

XiaqiongZhou-NOAA · 2024-05-13T18:14:01Z

Pointed to new Fv3atm.

…

On Mon, May 13, 2024 at 2:11 PM Brian Curtis ***@***.***> wrote: Looks like FV3ATM is supposed to be at: ***@***.*** <XiaqiongZhou-NOAA/fv3atm@fc2630c> This hash was after the indentation change I requested. — Reply to this email directly, view it on GitHub <#2255 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALQE7UQNTRKC5G6QUTWIYUDZCD65HAVCNFSM6AAAAABG22ZRWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBYGQ4TQNRTGU> . You are receiving this because you were mentioned.Message ID: ***@***.***>

DeniseWorthen · 2024-05-13T19:16:29Z

@zach1221 This PR is not ready for the commit Q unless all ORT tests are completed and passed. This test is a possible implementation target and must successfully pass all ORTs at the time of commit.

zach1221 · 2024-05-15T16:13:12Z

Disregard the Orion failure message above.

zach1221 · 2024-05-15T16:16:00Z

I'll follow up on the cubed-sphere sub-pr and we can begin the merge process.

DusanJovic-NOAA · 2024-05-15T16:29:23Z

The issue with Acorn was incorrectly put as cpld_control_c48 while it was really with cpld_warmstart_c48. I doubled the PE's for I/J to account for an issue with the test using too much memory.

cpld_warmstart_c48 should need approximately same amount of memory as cpld_control_c48? How did this test work before? Which change in this PR is causing this test to require more memory now?

BrianCurtis-NOAA · 2024-05-15T16:55:22Z

The issue with Acorn was incorrectly put as cpld_control_c48 while it was really with cpld_warmstart_c48. I doubled the PE's for I/J to account for an issue with the test using too much memory.

cpld_warmstart_c48 should need approximately same amount of memory as cpld_control_c48? How did this test work before? Which change in this PR is causing this test to require more memory now?

I had the same questions. Denise mentioned the same to me earlier as well. The run completed fully (as in generated all the sfcf and atmf files it needed) but right at the end it had this, and the comparisons were not run:

  Memuse(MB) at Memory HiWaterMark=  1.429E+02  1.683E+02  1.019E+01  1.508E+02
Application ca116624-ed06-4846-8cb0-deb5a0be1c61 resources: utime=1039s stime=1207s maxrss=2697772KB inblock=3180456 oublock=4777392 minflt=1725554 majflt=453 nvcsw=125660 nivcsw=6252

DusanJovic-NOAA · 2024-05-15T17:03:50Z

The issue with Acorn was incorrectly put as cpld_control_c48 while it was really with cpld_warmstart_c48. I doubled the PE's for I/J to account for an issue with the test using too much memory.

cpld_warmstart_c48 should need approximately same amount of memory as cpld_control_c48? How did this test work before? Which change in this PR is causing this test to require more memory now?

I had the same questions. Denise mentioned the same to me earlier as well. The run completed fully (as in generated all the sfcf and atmf files it needed) but right at the end it had this, and the comparisons were not run:
  Memuse(MB) at Memory HiWaterMark=  1.429E+02  1.683E+02  1.019E+01  1.508E+02
Application ca116624-ed06-4846-8cb0-deb5a0be1c61 resources: utime=1039s stime=1207s maxrss=2697772KB inblock=3180456 oublock=4777392 minflt=1725554 majflt=453 nvcsw=125660 nivcsw=6252

This does not indicate any error.

If the model finished successfully then the issue is not that the test is using too much memory. There must be some other reason why the comparisons were not run. Please revert to original setup and try again.

This reverts commit 7987b99.

on-behalf-of @ufs-community <[email protected]>

BrianCurtis-NOAA · 2024-05-15T18:13:51Z

====START OF ACORN REGRESSION TESTING LOG====

UFSWM hash used in testing:
16f3271ba7e5266662904d1524d87a80f97bf9bf

Submodule hashes used in testing:
 37cbb7d6840ae7515a9a8f0dfd4d89461b3396d1 AQM (v0.2.0-37-g37cbb7d)
 be5d28fd1b60522e6fc98aefeead20e6aac3530b AQM/src/model/CMAQ (CMAQv5.2.1_07Feb2018-198-gbe5d28fd1)
 3d7067a523b8557058755289e4275f5f5c985daf CDEPS-interface/CDEPS (cdeps0.4.17-40-g3d7067a)
 9c0cd8d3ba5963d01711094a3c81b6358fcb8367 CICE-interface/CICE (CICE6.0.0-448-g9c0cd8d)
 f6ff8f7c4d4cb6feabe3651b13204cf43fc948e3 CICE-interface/CICE/icepack (Icepack1.1.0-182-gf6ff8f7)
 4e19850cb083bc474b7cde5dc2f8506ec74cc442 CMEPS-interface/CMEPS (cmeps_v0.4.1-2306-g4e19850)
 cabd7753ae17f7bfcc6dad56daf10868aa51c3f4 CMakeModules (v1.0.0-28-gcabd775)
 b329c2ccc4c5c6936cb1f548eef83d2a1ee36ae2 FV3 (remotes/origin/hydro)
 aabfdccae440da7389195762601db7fcc09f6f11 FV3/atmos_cubed_sphere (remotes/origin/hydro)
 741212e4962d371520f773e2be9481142f79635e FV3/ccpp/framework (ccpp_transition_to_vlab_master_20190705-718-g741212e)
 1c2a898e97f069eeeb930cfb542db67ed349b72c FV3/ccpp/physics (EP4-744-g1c2a898e)
 74a0e098b2163425e4b5466c2dfcf8ae26d560a5 FV3/ccpp/physics/physics/Radiation/RRTMGP/rte-rrtmgp (v1.6)
 be0410ece28f2b5b9c089f8ca09ce0c80c79fe6c FV3/upp (upp_v10.2.0-191-gbe0410e)
-1ba8270870947b583cd51bc72ff8960f4c1fb36e FV3/upp/sorc/libIFI.fd
-7476b8f2790a47d788f79cebfdbb551567ae7cf8 FV3/upp/sorc/ncep_post.fd/post_gtg.fd
 041422934cae1570f2f0e67239d5d89f11c6e1b7 GOCART (sdr_v2.1.2.6-119-g0414229)
 35789c757766e07f688b4c0c7c5229816f224b09 HYCOM-interface/HYCOM (2.3.00-121-g35789c7)
 3bcfbbea8b865c3f88497049fde044c1145bd202 MOM6-interface/MOM6 (dev/master/repository_split_2014.10.10-10050-g3bcfbbea8)
 9423197f894112edfcb1502245f7d7b873d551f9 MOM6-interface/MOM6/pkg/CVMix-src (9423197)
 29e64d652786e1d076a05128c920f394202bfe10 MOM6-interface/MOM6/pkg/GSW-Fortran (29e64d6)
 1e259014c1eba9070cec7027d8b4b479ae54275a NOAHMP-interface/noahmp (v3.7.1-430-g1e25901)
 d9b3172f4197c65d471662c6952a668152d71230 WW3 (6.07.1-345-gd9b3172f)
 31e4e3e57bad8a7cdaa0ce2d3c4efbca63b5f74a stochastic_physics (ufs-v2.0.0-216-g31e4e3e)


NOTES:
[Times](Memory) are at the end of each compile/test in format [MM:SS](Size).
The first time is for the full script (prep+run+finalize).
The second time is specifically for the run phase.
Times/Memory will be empty for failed tests.

BASELINE DIRECTORY: /lfs/h2/emc/nems/noscrub/emc.nems/RT/NEMSfv3gfs/develop-20240503
COMPARISON DIRECTORY: /lfs/h2/emc/ptmp/brian.curtis/FV3_RT/rt_17831

RT.SH OPTIONS USED:
* (-a) - HPC PROJECT ACCOUNT: GFS-DEV
* (-n) - RUN SINGLE TEST: cpld_warmstart_c48
* (-e) - USE ECFLOW

PASS -- COMPILE 's2s_intel' [09:21, 09:05] ( 1 warnings 1 remarks )
PASS -- TEST 'cpld_warmstart_c48_intel' [54:48, 01:29](2702 MB)

SYNOPSIS:
Starting Date/Time: 20240515 17:55:41
Ending Date/Time: 20240515 18:10:27
Total Time: 00h:14m:53s
Compiles Completed: 1/1
Tests Completed: 1/1

NOTES:
A file 'test_changes.list' was generated but is empty.
If you are using this log as a pull request verification, please commit 'test_changes.list'.

Result: SUCCESS

====END OF ACORN REGRESSION TESTING LOG====

the E in PBS is not for "error" but "Exiting" and caused the ecflow job to terminate before finishing even when the model did finish

… for ecflow

…l into hydro

DusanJovic-NOAA · 2024-05-15T18:32:44Z

====START OF ACORN REGRESSION TESTING LOG====

UFSWM hash used in testing:
16f3271ba7e5266662904d1524d87a80f97bf9bf

Submodule hashes used in testing:
 37cbb7d6840ae7515a9a8f0dfd4d89461b3396d1 AQM (v0.2.0-37-g37cbb7d)
 be5d28fd1b60522e6fc98aefeead20e6aac3530b AQM/src/model/CMAQ (CMAQv5.2.1_07Feb2018-198-gbe5d28fd1)
 3d7067a523b8557058755289e4275f5f5c985daf CDEPS-interface/CDEPS (cdeps0.4.17-40-g3d7067a)
 9c0cd8d3ba5963d01711094a3c81b6358fcb8367 CICE-interface/CICE (CICE6.0.0-448-g9c0cd8d)
 f6ff8f7c4d4cb6feabe3651b13204cf43fc948e3 CICE-interface/CICE/icepack (Icepack1.1.0-182-gf6ff8f7)
 4e19850cb083bc474b7cde5dc2f8506ec74cc442 CMEPS-interface/CMEPS (cmeps_v0.4.1-2306-g4e19850)
 cabd7753ae17f7bfcc6dad56daf10868aa51c3f4 CMakeModules (v1.0.0-28-gcabd775)
 b329c2ccc4c5c6936cb1f548eef83d2a1ee36ae2 FV3 (remotes/origin/hydro)
 aabfdccae440da7389195762601db7fcc09f6f11 FV3/atmos_cubed_sphere (remotes/origin/hydro)
 741212e4962d371520f773e2be9481142f79635e FV3/ccpp/framework (ccpp_transition_to_vlab_master_20190705-718-g741212e)
 1c2a898e97f069eeeb930cfb542db67ed349b72c FV3/ccpp/physics (EP4-744-g1c2a898e)
 74a0e098b2163425e4b5466c2dfcf8ae26d560a5 FV3/ccpp/physics/physics/Radiation/RRTMGP/rte-rrtmgp (v1.6)
 be0410ece28f2b5b9c089f8ca09ce0c80c79fe6c FV3/upp (upp_v10.2.0-191-gbe0410e)
-1ba8270870947b583cd51bc72ff8960f4c1fb36e FV3/upp/sorc/libIFI.fd
-7476b8f2790a47d788f79cebfdbb551567ae7cf8 FV3/upp/sorc/ncep_post.fd/post_gtg.fd
 041422934cae1570f2f0e67239d5d89f11c6e1b7 GOCART (sdr_v2.1.2.6-119-g0414229)
 35789c757766e07f688b4c0c7c5229816f224b09 HYCOM-interface/HYCOM (2.3.00-121-g35789c7)
 3bcfbbea8b865c3f88497049fde044c1145bd202 MOM6-interface/MOM6 (dev/master/repository_split_2014.10.10-10050-g3bcfbbea8)
 9423197f894112edfcb1502245f7d7b873d551f9 MOM6-interface/MOM6/pkg/CVMix-src (9423197)
 29e64d652786e1d076a05128c920f394202bfe10 MOM6-interface/MOM6/pkg/GSW-Fortran (29e64d6)
 1e259014c1eba9070cec7027d8b4b479ae54275a NOAHMP-interface/noahmp (v3.7.1-430-g1e25901)
 d9b3172f4197c65d471662c6952a668152d71230 WW3 (6.07.1-345-gd9b3172f)
 31e4e3e57bad8a7cdaa0ce2d3c4efbca63b5f74a stochastic_physics (ufs-v2.0.0-216-g31e4e3e)


NOTES:
[Times](Memory) are at the end of each compile/test in format [MM:SS](Size).
The first time is for the full script (prep+run+finalize).
The second time is specifically for the run phase.
Times/Memory will be empty for failed tests.

BASELINE DIRECTORY: /lfs/h2/emc/nems/noscrub/emc.nems/RT/NEMSfv3gfs/develop-20240503
COMPARISON DIRECTORY: /lfs/h2/emc/ptmp/brian.curtis/FV3_RT/rt_17831

RT.SH OPTIONS USED:
* (-a) - HPC PROJECT ACCOUNT: GFS-DEV
* (-n) - RUN SINGLE TEST: cpld_warmstart_c48
* (-e) - USE ECFLOW

PASS -- COMPILE 's2s_intel' [09:21, 09:05] ( 1 warnings 1 remarks )
PASS -- TEST 'cpld_warmstart_c48_intel' [54:48, 01:29](2702 MB)

SYNOPSIS:
Starting Date/Time: 20240515 17:55:41
Ending Date/Time: 20240515 18:10:27
Total Time: 00h:14m:53s
Compiles Completed: 1/1
Tests Completed: 1/1

NOTES:
A file 'test_changes.list' was generated but is empty.
If you are using this log as a pull request verification, please commit 'test_changes.list'.

Result: SUCCESS

====END OF ACORN REGRESSION TESTING LOG====

the E in PBS is not for "error" but "Exiting" and caused the ecflow job to terminate before finishing even when the model did finish

Hmm. This was never an issue. Before the recent 'cleanup' code that handled the 'E' status looked like:

      elif [[ $status = 'E' ]] || [[ $status = 'C' ]] || [[ $status = '-' ]];  then
        status_label='finished'
        test_status='DONE'
        exit_status=$( qstat ${jobid} -x -f | grep Exit_status | awk '{print $3}')
        if [[ $exit_status != 0 ]]; then
          test_status='FAIL'
        fi
      else
        status_label='finished'
      fi

That part of code has been completely rewritten. I wonder how many more subtle error like this we are going to find in the future.

For example the code that checks the 'Exit_status' of the finished job:

     exit_status=$( qstat ${jobid} -x -f | grep Exit_status | awk '{print $3}')
        if [[ $exit_status != 0 ]]; then
          test_status='FAIL'
        fi

is completely removed. Why?

DusanJovic-NOAA · 2024-05-15T18:47:43Z

Also, this logic has been removed:

      else
        echo "Slurm unknown status ${status}. Check sacct ..."
        sacct -n -j ${slurm_id} --format=JobID,state%20,Jobname%20
        status_label=$( sacct -n -j ${slurm_id} --format=JobID,state%20,Jobname%20 | grep "^${slurm_id}" | grep ${JBNME} | awk '{print $2}' )
        if [[ $status_label = 'FAILED' ]] || [[ $status_label = 'TIMEOUT' ]] || [[ $status_label = 'CANCELLED' ]] ; then
            test_status='FAIL'
        fi
      fi

BrianCurtis-NOAA · 2024-05-15T18:52:35Z

@DusanJovic-NOAA I don't think any of this pertains to this PR, we can move forward with this PR at this point.

Please feel free to start an e-mail discussion on recent rt.sh and rt_utils.sh changes.

zach1221 · 2024-05-15T20:38:07Z

@XiaqiongZhou-NOAA Fv3atm merged. Please update submodule hash and revert change to .gitmodule url.
Hash: NOAA-EMC/fv3atm@9534e14

XiaqiongZhou-NOAA · 2024-05-15T20:40:29Z

done.

…

On Wed, May 15, 2024 at 4:38 PM zach1221 ***@***.***> wrote: @XiaqiongZhou-NOAA <https://github.com/XiaqiongZhou-NOAA> Fv3atm merged. Please update submodule hash and revert change to .gitmodule url. Hash: ***@***.*** <NOAA-EMC/fv3atm@9534e14> — Reply to this email directly, view it on GitHub <#2255 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALQE7UX2AAEHTIACLDPFGVTZCPBULAVCNFSM6AAAAABG22ZRWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJTGQYTMNJYGE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

zach1221 · 2024-05-15T20:45:00Z

@DeniseWorthen @NickSzapiro-NOAA can you please review your unresolved conversation?

NickSzapiro-NOAA · 2024-05-15T21:11:53Z

@DeniseWorthen @NickSzapiro-NOAA can you please review your unresolved conversation?

@zach1221 I marked my comment as resolved. I don't see any other options... This PR still needs commit messages, fwiw

zach1221 · 2024-05-15T22:29:30Z

@XiaqiongZhou-NOAA I think Fernando mentioned this previously, but can you please fill out the commit message section of the PR template, which summarizes briefly your code changes included in this PR and sub-component PRs?

XiaqiongZhou-NOAA · 2024-05-15T23:01:20Z

@XiaqiongZhou-NOAA I think Fernando mentioned this previously, but can you please fill out the commit message section of the PR template, which summarizes briefly your code changes included in this PR and sub-component PRs?

Sorry, I did not notice. Commit message is added.

XiaqiongZhou-NOAA added 4 commits April 24, 2024 16:30

Point to personal branch

1025aad

Add a RT for SFS with hydrostatic options

7c47ff2

Merge remote-tracking branch 'ufs-community/develop' into hydro

a7c1346

Add test_changes.list based on the tests on Hera

d3fe852

This was referenced Apr 26, 2024

Change compiling flag for hydrostatic options NOAA-EMC/fv3atm#828

Merged

Remove -DUSE_COND and -DMOIST_CAPPA for hydrostatic runs NOAA-GFDL/GFDL_atmos_cubed_sphere#337

Merged

XiaqiongZhou-NOAA marked this pull request as ready for review April 26, 2024 14:36

update FV3

f44f6d6

XiaqiongZhou-NOAA mentioned this pull request May 1, 2024

Write out HGT for hydrostatic run and omega from the model output NOAA-EMC/UPP#946

Closed

XiaqiongZhou-NOAA added 2 commits May 10, 2024 01:33

update FV3

2fba68f

Merge ufs-weather-model/develop

b2c84dc

NickSzapiro-NOAA reviewed May 10, 2024

View reviewed changes

tests/rt.conf Outdated Show resolved Hide resolved

rename the RT

6a8403b

Merge remote-tracking branch 'ufs-community/develop' into hydro

6c91578

zach1221 added Baseline Updates Current baselines will be updated. Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. labels May 13, 2024

zach1221 removed the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label May 13, 2024

update FV3

b4cec5d

zach1221 added the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label May 13, 2024

Update FV3

a029cee

zach1221 added Baseline Updates Current baselines will be updated. Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. labels May 15, 2024

Delete tests/logs/RT-run-Orion.log

c1f4dfc

BrianCurtis-NOAA and others added 3 commits May 15, 2024 17:05

Revert "Acorn memory issue with cpld_control_c48, doubled IN/JN PES"

16f3271

This reverts commit 7987b99.

[AutoRT] Orion Job Completed.

cb5fb76

on-behalf-of @ufs-community <[email protected]>

revert Orion logs: all passed

bb1b216

BrianCurtis-NOAA added 2 commits May 15, 2024 18:14

Adjust E to be Exiting for PBS and not 'Error' in rt_utils.sh nalding…

87603cb

… for ecflow

Merge branch 'hydro' of github.com:XiaqiongZhou-NOAA/ufs-weather-mode…

b24cc42

…l into hydro

Point to NOAA-EMC/fv3atm

7b5254c

zach1221 requested review from jkbk2004 and BrianCurtis-NOAA May 15, 2024 20:42

jkbk2004 approved these changes May 15, 2024

View reviewed changes

NickSzapiro-NOAA approved these changes May 15, 2024

View reviewed changes

BrianCurtis-NOAA approved these changes May 15, 2024

View reviewed changes

FernandoAndrade-NOAA merged commit 1c6b4d4 into ufs-community:develop May 16, 2024
3 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add a RT for SFS with hydrostatic options #2255

Add a RT for SFS with hydrostatic options #2255

XiaqiongZhou-NOAA commented Apr 26, 2024 •

edited

Loading

zach1221 commented May 10, 2024

BrianCurtis-NOAA commented May 10, 2024

JessicaMeixner-NOAA commented May 10, 2024

NickSzapiro-NOAA commented May 10, 2024

zach1221 commented May 11, 2024

zach1221 commented May 13, 2024

XiaqiongZhou-NOAA commented May 13, 2024 via email

BrianCurtis-NOAA commented May 13, 2024

XiaqiongZhou-NOAA commented May 13, 2024 via email

DeniseWorthen commented May 13, 2024

zach1221 commented May 15, 2024

zach1221 commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

BrianCurtis-NOAA commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

BrianCurtis-NOAA commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

BrianCurtis-NOAA commented May 15, 2024

zach1221 commented May 15, 2024

XiaqiongZhou-NOAA commented May 15, 2024 via email

zach1221 commented May 15, 2024

NickSzapiro-NOAA commented May 15, 2024

zach1221 commented May 15, 2024

XiaqiongZhou-NOAA commented May 15, 2024

Add a RT for SFS with hydrostatic options #2255

Add a RT for SFS with hydrostatic options #2255

Conversation

XiaqiongZhou-NOAA commented Apr 26, 2024 • edited Loading

Commit Queue Requirements:

Description:

Commit Message:

Priority:

Sub component Pull Requests:

Changes

Regression Test Changes (Please commit test_changes.list):

Input data Changes:

Library Changes/Upgrades:

Testing Log:

zach1221 commented May 10, 2024

BrianCurtis-NOAA commented May 10, 2024

JessicaMeixner-NOAA commented May 10, 2024

NickSzapiro-NOAA commented May 10, 2024

zach1221 commented May 11, 2024

zach1221 commented May 13, 2024

XiaqiongZhou-NOAA commented May 13, 2024 via email

BrianCurtis-NOAA commented May 13, 2024

XiaqiongZhou-NOAA commented May 13, 2024 via email

DeniseWorthen commented May 13, 2024

zach1221 commented May 15, 2024

zach1221 commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

BrianCurtis-NOAA commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

BrianCurtis-NOAA commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

DusanJovic-NOAA commented May 15, 2024

BrianCurtis-NOAA commented May 15, 2024

zach1221 commented May 15, 2024

XiaqiongZhou-NOAA commented May 15, 2024 via email

zach1221 commented May 15, 2024

NickSzapiro-NOAA commented May 15, 2024

zach1221 commented May 15, 2024

XiaqiongZhou-NOAA commented May 15, 2024

XiaqiongZhou-NOAA commented Apr 26, 2024 •

edited

Loading