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

Failure in regression test setup scripts not caught #7

Closed
climbfuji opened this issue Nov 5, 2019 · 2 comments
Closed

Failure in regression test setup scripts not caught #7

climbfuji opened this issue Nov 5, 2019 · 2 comments

Comments

@climbfuji
Copy link
Collaborator

climbfuji commented Nov 5, 2019

If the rt_fv3.sh script exits with a failure, for example if an input namelist template for creating the model namelist is not found, this error is not caught even though the job is never submitted. For regression tests that do not compare files (such as the tests in debug mode), this leads to rt.sh reporting that the test succeeded.

This is because of the following lines in tests/run_test.sh:

...
# Submit the actual test run script
echo "Test ${TEST_NR} ${TEST_NAME} ${TEST_DESCR}"
trap 'echo "run_test.sh: Test ${TEST_NAME} killed"; kill $(jobs -p); wait; trap 0; exit' 1 2 3 4 5 6 7 8 10 12 13 15
./${RUN_SCRIPT} > ${RUNDIR_ROOT}/${TEST_NAME}${RT_SUFFIX}.log 2>&1 &
wait
...
@climbfuji
Copy link
Collaborator Author

One way to fix this is

# Submit the actual test run script
echo "Test ${TEST_NR} ${TEST_NAME} ${TEST_DESCR}"
trap 'echo "run_test.sh: Test ${TEST_NAME} killed"; kill $(jobs -p); wait; trap 0; exit' 1 2 3 4 5 6 7 8 10 12 13 15
./${RUN_SCRIPT} > ${RUNDIR_ROOT}/${TEST_NAME}${RT_SUFFIX}.log 2>&1
...

which, in the event of an error in tests/rt_fv3.sh, leads to

++ echo 'rt.sh error on line 639'
rt.sh error on line 639
++ rt_trap
++ [[ false == true ]]
++ [[ false == true ]]
++ cleanup
++ rm -rf /scratch1/BMC/gmtb/Dom.Heinzeller/NEMSfv3gfs/ufs-weather-model_dtc-develop_20191105/regtestfailure/tests/lock
++ trap 0
++ exit

but I haven't tested this for the full set of regression tests (rt.conf).

climbfuji referenced this issue in climbfuji/ufs-weather-model Nov 26, 2019
dtc/develop: update submodule pointer for fv3atm 2019/11/25
@climbfuji
Copy link
Collaborator Author

climbfuji commented Dec 30, 2019

The issue described above is addressed in #18 for the "develop" branch, and in #25 for the "ufs_public_release" branch.

climbfuji referenced this issue in climbfuji/ufs-weather-model Feb 13, 2020
gsd/develop: update from from develop 2020/01/27
climbfuji referenced this issue in climbfuji/ufs-weather-model Jun 1, 2020
* Cleanup regression test scripts (remove moab, merge sbatch with slurm)
* Update stampede.intel build config, remove cheyenne.pgi config
* Remove outdated/old documentation in doc/README_xyz.txt
XiaSun-Atmos pushed a commit to XiaSun-Atmos/ufs-weather-model that referenced this issue Aug 8, 2020
Bugfixes in inline post, netcdf time stamp and gfs physics for coupled system. GFS physics update to reduce the cold bias in lower atmosphere layer. VLAB tickets: #69814 and #69735. fv3atm issue: ufs-community#4, ufs-community#5, ufs-community#6 and ufs-community#7.
DeniseWorthen referenced this issue in DeniseWorthen/ufs-weather-model Feb 25, 2021
mlee03 pushed a commit to mlee03/ufs-weather-model that referenced this issue Mar 1, 2021
junwang-noaa pushed a commit that referenced this issue Mar 9, 2021
…bm_ic directory out of inputdata directory; use aws ec2 for CI test; auto-rt fixes (#426)

* Modifications to run tests on aws ec2 instances
* Change layout numbers. Include all utests (#7)
* Remove unnecessary sleep in build_test.yml and buidl_status_check.py
* Increase sleep period in build_status_check.py to meet 60 requests/hr
* Add sleep back to build_status.check.py
* Minor changes. Change CI to develop branch
* update diag_tables
*add config variable for diag_table
*set up diag_table templates for benchmarks (pre/v16) versions
* fix diag table variable for v16 tests
* add 35 v16 bmark frac test
* add missing diag_table in bmark restart tests
* Move bm_ic directory out of input-data directory
* Replace sys exit with valueError in build_status_check.py
* updates for v16 stability
* remove un-used dz_min=6 parameter setting in bmark v16 tests
* add psm_bc variable and set to value of 1 (default 0) for v16 tests
* set dt_atmos=225 in v16 tests
* updates for dated BM_IC directory
* add INPUTDATA_ROOT_BMIC to rt.sh
* remove BM_IC variable in cpld_bmark_run.IN and use explict path to specify IC sources for ATM,MOM6,CICE5 and WW3 (only for 35d test)
* add variables in parm/input.benchmark_v16.nml.IN
* set dddmp = 0.2 and FSICL to 0 in v16 bmark tests
* modify v16 tests
* keep FSICL at default value for now
* reduce time for v16 wave test to 6hr
* update RTPWD to 20130308; skip-ci
* fix cpld_bmarkfrac_wave_v16 test
* fix timestamps on files for comparison to baseline after switching to 6hour test length
* remove restart file from WW3 comparison. Currently the restart file writes out only at hour 12. This is set in ww3_multi.inp. A change to allow writing a restart at hour 6 would require an update to the ww3 input data directory

Co-authored-by: MinsukJi-NOAA <[email protected]>
Co-authored-by: Brian Curtis <[email protected]>
Co-authored-by: climbfuji <[email protected]>
LarissaReames-NOAA pushed a commit to LarissaReames-NOAA/ufs-weather-model that referenced this issue Oct 22, 2021
LarissaReames-NOAA pushed a commit to LarissaReames-NOAA/ufs-weather-model that referenced this issue Oct 22, 2021
- bugfix for GNU compiler
- reduce noise in stdout

Squashed commit of the following:

commit 86c127f5b810b87004aeed3a625f9445d469c27e
Merge: ce66d6a f9bbb4f
Author: Dom Heinzeller <[email protected]>
Date:   Fri Apr 10 11:32:21 2020 -0600

    Merge pull request ufs-community#11 from climbfuji/update_dtc_develop_from_dev_emc_20200409

    Update dtc/develop from dev/emc 2020/04/09

commit f9bbb4f12e032378050cef4b7e5ac4b65a16bb70
Author: Dom Heinzeller <[email protected]>
Date:   Thu Apr 9 20:14:24 2020 -0600

    Bugfix for GNU compiler in model/fv_regional_bc.F90, reduce verbosity in driver/fvGFS/atmosphere.F90

commit ecca9978802ec9cc39dc114fd7363386f7ad9422
Merge: ce66d6a 80ce8ce
Author: Dom Heinzeller <[email protected]>
Date:   Thu Apr 9 09:38:48 2020 -0600

    Merge branch 'dev/emc' of https://github.com/NOAA-EMC/GFDL_atmos_cubed_sphere into HEAD

commit ce66d6a56e89b6330a2bef763755c7154047d829
Merge: 4279bf7 9bacf55
Author: Dom Heinzeller <[email protected]>
Date:   Fri Mar 13 13:00:41 2020 -0600

    Merge pull request ufs-community#10 from climbfuji/update_from_dev_emc_20200312_and_other_changes

    Update from dev/emc 2020/03/12

commit 9bacf554ed078dfa8c09ee662e7608442288e2d5
Merge: 4279bf7 371a29a
Author: Dom Heinzeller <[email protected]>
Date:   Thu Mar 12 10:33:44 2020 -0600

    Merge branch 'dev/emc' of https://github.com/NOAA-EMC/GFDL_atmos_cubed_sphere into HEAD

commit 4279bf78eb4ae0bad908eb991e3fb216018b7aea
Merge: 846f5a5 4858d33
Author: Dom Heinzeller <[email protected]>
Date:   Mon Mar 9 12:41:51 2020 -0600

    Merge pull request ufs-community#9 from climbfuji/update_dtc_develop_from_emc

    Update dtc/develop from dev/emc 2020/03/04

commit 4858d33838adcd87c1cd000ab4737dc0e7e56731
Merge: 846f5a5 db3acfb
Author: Dom Heinzeller <[email protected]>
Date:   Fri Feb 28 08:46:55 2020 -0700

    Merge branch 'dev/emc' of https://github.com/NOAA-EMC/GFDL_atmos_cubed_sphere into HEAD

commit 846f5a54b0764db1478c9aca49a74bd38322d9ad
Merge: 3a4dfd8 9a290ee
Author: Dom Heinzeller <[email protected]>
Date:   Mon Feb 3 07:43:08 2020 -0700

    Merge pull request ufs-community#7 from climbfuji/update_dtc_develop_from_dev-emc

    dtc/develop: update from dev/emc 2020/01/27

commit 9a290ee5f3307a9d7b02762fe6c29a1f5ccd7f55
Merge: 3a4dfd8 a56907a
Author: Dom Heinzeller <[email protected]>
Date:   Mon Jan 27 09:28:57 2020 -0700

    Merge branch 'dev/emc' of https://github.com/noaa-emc/GFDL_atmos_cubed_sphere into HEAD

commit 3a4dfd8c6c4ceb8cec06397f25cb229ecd98065b
Merge: 0c9ab9e 68576a6
Author: Dom Heinzeller <[email protected]>
Date:   Tue Dec 3 15:19:24 2019 -0700

    Merge pull request ufs-community#6 from climbfuji/dtc_develop_udpate_from_emc_20191127

    dtc/develop: update from EMC 2019/11/27

commit 68576a61f2a57236931e5ef1b8bd73a4256b0a5f
Merge: 0c9ab9e 452333a
Author: Dom Heinzeller <[email protected]>
Date:   Wed Nov 27 14:13:08 2019 -0700

    Merge branch 'dev/emc' of https://github.com/NOAA-EMC/GFDL_atmos_cubed_sphere into HEAD

commit 0c9ab9e3dfe6cda6fc78315b86a107a369756024
Merge: 9871607 b280b37
Author: Dom Heinzeller <[email protected]>
Date:   Fri Nov 22 16:17:57 2019 -0700

    Merge pull request ufs-community#5 from mzhangw/HAFS_fer_hires

    minimal changes to make nwat =4 compatible with Ferrier-Aligo MP scheme

commit b280b373b421fcb9d84bd1196a40ec56fcf279ff
Author: Man.Zhang <[email protected]>
Date:   Tue Nov 19 14:53:57 2019 -0700

    bug fix

commit 98c58ad3d0444859aae54c31bc2b86ba30b4a9ac
Author: Man.Zhang <[email protected]>
Date:   Tue Nov 19 14:35:10 2019 -0700

    bug fix

commit 8805acd93152bb76d5c4d0900f314631f549fbe2
Author: Man.Zhang <[email protected]>
Date:   Tue Nov 19 14:16:22 2019 -0700

    From Chunxi: The file fv_mapz.F90 also needs to be modified (K_warm)

commit f8a257a922a9047e98c176affec054e2a4d9c6a9
Author: Man.Zhang <[email protected]>
Date:   Thu Nov 14 19:12:39 2019 -0700

    use upper-case CCPP

commit 77f4fbad9311ad406c6486ad0912069de3085f07
Author: Man.Zhang <[email protected]>
Date:   Thu Nov 14 13:40:59 2019 -0700

     implement FA scheme water loading option if nwat =4

commit 987160799b3a1c0f11de9883843f20421eb03fe8
Merge: ed75004 786447c
Author: Dom Heinzeller <[email protected]>
Date:   Tue Nov 5 06:59:34 2019 +0900

    Merge branch 'dev/emc' of https://github.com/NOAA-EMC/GFDL_atmos_cubed_sphere into HEAD

commit ed75004634374b02f8b222dd873190a91126ce4f
Merge: 1d6035a f627bfb
Author: Dom Heinzeller <[email protected]>
Date:   Sat Oct 12 08:45:14 2019 +0900

    Merge pull request ufs-community#4 from climbfuji/update_gmtb_develop_with_vlab_master_20191006

    Update NCAR gmtb/develop with NOAA-EMC dev/emc 2019/10/06

commit f627bfb177dc1f2d0bd10371540c43c51ce8eec4
Author: Dom Heinzeller <[email protected]>
Date:   Mon Oct 7 12:12:24 2019 +0900

    Add missing change in fv_cmp.F90 - now identical with EMC code

commit e868186fd7d64fc2eeb7ec2c59e88c889e62fc93
Author: Dusan Jovic <[email protected]>
Date:   Tue Sep 24 01:57:11 2019 +0000

    change delz from positive value to the original value in the model

commit 672eb0e8d67c008437724681250e7aa8aa81ec31
Author: fanglin.yang <[email protected]>
Date:   Thu Sep 5 19:24:09 2019 +0000

        VLab Issue #68141

        modified:   docs/FV3_citations.bib
        modified:   model/fv_cmp.F90

        1) Add an option (namelist parameter: intqs) to use temperature instead of the liquid frozen
        temperature to calculate saturation mixing ratio in deriving GFDLMP PDF cloud cover.
        2) Add a literature reference.

commit 1d6035ab053a6b5257b32dd9b4a49faa7ee4c78d
Merge: 94ab0e1 fec8205
Author: Dom Heinzeller <[email protected]>
Date:   Mon Sep 23 09:32:55 2019 -0600

    Merge pull request ufs-community#2 from climbfuji/remove_transition_mode

    Remove TRANSITION mode

commit fec8205b9418dea28de4e2c9f05f347f3cf460d2
Author: climbfuji <[email protected]>
Date:   Fri Sep 20 08:52:01 2019 -0600

    Remove TRANSITION mode

commit 94ab0e1317ffa50279dd6269a462999a2bea15e7
Author: Jun.Wang <[email protected]>
Date:   Mon Aug 26 21:03:55 2019 +0000

    add option to output omega

commit b700cae673038f82189a1e6e0cb48a10a58ea726
Author: climbfuji <[email protected]>
Date:   Fri Aug 30 11:58:00 2019 -0600

    model/fv_dynamics.F90: replicating previous change to gmtb/develop branch of FV3, always allocate dtdt_m for IPD version

commit 3df11713e0ba9299d95512476a9ee784048629c9
Author: Laurie Carson <[email protected]>
Date:   Fri Aug 16 10:46:56 2019 -0600

    Add codeowners for NCAR fork
@dustinswales dustinswales mentioned this issue Dec 7, 2021
16 tasks
epic-cicd-jenkins pushed a commit that referenced this issue Apr 17, 2023
 - Adds a CMake build using ExternalProject_Add.
 - Makes modifications to build_all.sh and install_all.sh for back-compatibility with old build system. The old build system will remain in, but is now deprecated, and will be removed in the coming weeks.

To build, have the necessary modules loaded and run:

```
mkdir build
cd build
cmake .. -DCMAKE_INSTALL_PREFIX=<prefix>
make
```

The docs/ directory contains README files with instructions for specific platforms/compilers

 - **Cheyenne**: Build and end-to-end test ("DOT_OR_USCORE" test case) was successful on Cheyenne with intel, both for the cmake build and the old build script (that will soon be deprecated). Path to tests: /glade/scratch/kavulich/UFS_CAM/testing/SRW_PR_27/expt_dirs/
 - **Hera**: Build and end-to-end tests successful (aside from expected failures). Path to tests: /scratch2/BMC/det/kavulich/workdir/SRW_PR_27/expt_dirs
 - **Jet**: Build test was successful.

Resolves #7, partially resolves #5

Co-authored-by: kgerheiser <[email protected]>; Michael Kavulich, Jr <[email protected]>
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

No branches or pull requests

1 participant