Skip to content

fix grv function used by NST#2001

Merged
zach1221 merged 14 commits into
ufs-community:developfrom
DeniseWorthen:bugfix/grvfunction
Nov 20, 2023
Merged

fix grv function used by NST#2001
zach1221 merged 14 commits into
ufs-community:developfrom
DeniseWorthen:bugfix/grvfunction

Conversation

@DeniseWorthen
Copy link
Copy Markdown
Collaborator

@DeniseWorthen DeniseWorthen commented Nov 15, 2023

PR Author Checklist:

  • I have linked PR's from all sub-components involved in section below.
  • I am confirming reviews are completed in ALL sub-component PR's.
  • I have run the full RT suite on either Hera/Cheyenne AND have attached the log to this PR below this line:
  • I have added the list of all failed regression tests to "Anticipated changes" section.
  • I have filled out all sections of the template.

Description

Fixes the argument used by the grv function.

Linked Issues and Pull Requests

Associated UFSWM Issue to close

Subcomponent Pull Requests

Blocking Dependencies

Subcomponents involved:

  • AQM
  • CDEPS
  • CICE
  • CMEPS
  • CMakeModules
  • FV3
  • GOCART
  • HYCOM
  • MOM6
  • NOAHMP
  • WW3
  • stochastic_physics
  • none

Anticipated Changes

Input data

  • No changes are expected to input data.
  • Changes are expected to input data:
    • New input data.
    • Updated input data.

Regression Tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
Tests effected by changes in this PR:

Few tests pass. They include hrrr tests, datm_cdeps tests and some hafs tests.

Three tests timed out on the initial test, but ran successfully when retested. The three tests which timed-out were

Test 115 conus13km_debug_intel FAIL
Test 129 hafs_regional_specified_moving_1nest_atm_intel FAIL
Test 133 hafs_regional_storm_following_1nest_atm_ocn_debug_intel FAIL

The following tests failed with non-B4B comparisons.

Test 001 cpld_control_p8_mixedmode_intel FAIL
Test 002 cpld_control_p8_intel FAIL
Test 004 cpld_control_qr_p8_intel FAIL
Test 006 cpld_2threads_p8_intel FAIL
Test 007 cpld_decomp_p8_intel FAIL
Test 008 cpld_mpi_p8_intel FAIL
Test 009 cpld_control_ciceC_p8_intel FAIL
Test 010 cpld_control_c192_p8_intel FAIL
Test 012 cpld_bmark_p8_intel FAIL
Test 014 cpld_control_noaero_p8_intel FAIL
Test 015 cpld_control_nowave_noaero_p8_intel FAIL
Test 016 cpld_debug_p8_intel FAIL
Test 017 cpld_debug_noaero_p8_intel FAIL
Test 019 cpld_control_c48_intel FAIL
Test 020 cpld_control_p8_faster_intel FAIL
Test 021 control_flake_intel FAIL
Test 022 control_CubedSphereGrid_intel FAIL
Test 023 control_CubedSphereGrid_parallel_intel FAIL
Test 024 control_latlon_intel FAIL
Test 025 control_wrtGauss_netcdf_parallel_intel FAIL
Test 026 control_c48_intel FAIL
Test 027 control_c192_intel FAIL
Test 028 control_c384_intel FAIL
Test 029 control_c384gdas_intel FAIL
Test 030 control_stochy_intel FAIL
Test 032 control_lndp_intel FAIL
Test 033 control_iovr4_intel FAIL
Test 034 control_iovr5_intel FAIL
Test 035 control_p8_intel FAIL
Test 036 control_p8_ugwpv1_intel FAIL
Test 038 control_noqr_p8_intel FAIL
Test 040 control_decomp_p8_intel FAIL
Test 041 control_2threads_p8_intel FAIL
Test 042 control_p8_lndp_intel FAIL
Test 043 control_p8_rrtmgp_intel FAIL
Test 044 control_p8_mynn_intel FAIL
Test 045 merra2_thompson_intel FAIL
Test 046 regional_control_intel FAIL
Test 048 regional_decomp_intel FAIL
Test 049 regional_2threads_intel FAIL
Test 050 regional_noquilt_intel FAIL
Test 051 regional_netcdf_parallel_intel FAIL
Test 052 regional_2dwrtdecomp_intel FAIL
Test 053 regional_wofs_intel FAIL
Test 054 rap_control_intel FAIL
Test 056 rap_decomp_intel FAIL
Test 057 rap_2threads_intel FAIL
Test 059 rap_sfcdiff_intel FAIL
Test 060 rap_sfcdiff_decomp_intel FAIL
Test 066 rrfs_v1beta_intel FAIL
Test 067 rrfs_v1nssl_intel FAIL
Test 068 rrfs_v1nssl_nohailnoccn_intel FAIL
Test 069 control_csawmg_intel FAIL
Test 070 control_csawmgt_intel FAIL
Test 071 control_ras_intel FAIL
Test 073 control_p8_faster_intel FAIL
Test 074 regional_control_faster_intel FAIL
Test 075 control_CubedSphereGrid_debug_intel FAIL
Test 076 control_wrtGauss_netcdf_parallel_debug_intel FAIL
Test 077 control_stochy_debug_intel FAIL
Test 078 control_lndp_debug_intel FAIL
Test 079 control_csawmg_debug_intel FAIL
Test 080 control_csawmgt_debug_intel FAIL
Test 081 control_ras_debug_intel FAIL
Test 082 control_diag_debug_intel FAIL
Test 083 control_debug_p8_intel FAIL
Test 084 regional_debug_intel FAIL
Test 085 rap_control_debug_intel FAIL
Test 089 rap_unified_drag_suite_debug_intel FAIL
Test 090 rap_diag_debug_intel FAIL
Test 091 rap_cires_ugwp_debug_intel FAIL
Test 092 rap_unified_ugwp_debug_intel FAIL
Test 093 rap_lndp_debug_intel FAIL
Test 094 rap_progcld_thompson_debug_intel FAIL
Test 095 rap_noah_debug_intel FAIL
Test 096 rap_sfcdiff_debug_intel FAIL
Test 097 rap_noah_sfcdiff_cires_ugwp_debug_intel FAIL
Test 098 rrfs_v1beta_debug_intel FAIL
Test 099 rap_clm_lake_debug_intel FAIL
Test 100 rap_flake_debug_intel FAIL
Test 102 rap_control_dyn32_phy32_intel FAIL
Test 104 rap_2threads_dyn32_phy32_intel FAIL
Test 112 rap_control_dyn64_phy32_intel FAIL
Test 113 rap_control_debug_dyn32_phy32_intel FAIL
Test 115 conus13km_debug_intel FAIL
Test 119 rap_control_dyn64_phy32_debug_intel FAIL
Test 120 hafs_regional_atm_intel FAIL
Test 121 hafs_regional_atm_thompson_gfdlsf_intel FAIL
Test 123 hafs_regional_atm_wav_intel FAIL
Test 125 hafs_regional_1nest_atm_intel FAIL
Test 126 hafs_regional_telescopic_2nests_atm_intel FAIL
Test 127 hafs_global_1nest_atm_intel FAIL
Test 128 hafs_global_multiple_4nests_atm_intel FAIL
Test 129 hafs_regional_specified_moving_1nest_atm_intel FAIL
Test 130 hafs_regional_storm_following_1nest_atm_intel FAIL
Test 132 hafs_global_storm_following_1nest_atm_intel FAIL
Test 133 hafs_regional_storm_following_1nest_atm_ocn_debug_intel FAIL
Test 155 control_p8_atmlnd_sbs_intel FAIL
Test 156 atmwav_control_noaero_p8_intel FAIL
Test 157 control_atmwav_intel FAIL
Test 158 atmaero_control_p8_intel FAIL
Test 159 atmaero_control_p8_rad_intel FAIL
Test 160 atmaero_control_p8_rad_micro_intel FAIL
Test 161 regional_atmaq_intel FAIL
Test 162 regional_atmaq_debug_intel FAIL
Test 163 regional_atmaq_faster_intel FAIL
Test 164 control_c48_gnu FAIL
Test 165 control_stochy_gnu FAIL
Test 166 control_ras_gnu FAIL
Test 167 control_p8_gnu FAIL
Test 168 control_p8_ugwpv1_gnu FAIL
Test 169 control_flake_gnu FAIL
Test 170 rap_control_gnu FAIL
Test 171 rap_decomp_gnu FAIL
Test 172 rap_2threads_gnu FAIL
Test 174 rap_sfcdiff_gnu FAIL
Test 175 rap_sfcdiff_decomp_gnu FAIL
Test 183 rrfs_v1beta_gnu FAIL
Test 184 control_diag_debug_gnu FAIL
Test 185 regional_debug_gnu FAIL
Test 186 rap_control_debug_gnu FAIL
Test 190 rap_diag_debug_gnu FAIL
Test 191 rap_noah_sfcdiff_cires_ugwp_debug_gnu FAIL
Test 192 rap_progcld_thompson_debug_gnu FAIL
Test 193 rrfs_v1beta_debug_gnu FAIL
Test 194 control_ras_debug_gnu FAIL
Test 195 control_stochy_debug_gnu FAIL
Test 196 control_debug_p8_gnu FAIL
Test 197 rap_flake_debug_gnu FAIL
Test 198 rap_clm_lake_debug_gnu FAIL
Test 199 rap_control_dyn32_phy32_gnu FAIL
Test 201 rap_2threads_dyn32_phy32_gnu FAIL
Test 209 rap_control_dyn64_phy32_gnu FAIL
Test 210 rap_control_debug_dyn32_phy32_gnu FAIL
Test 216 rap_control_dyn64_phy32_debug_gnu FAIL
Test 217 cpld_control_p8_gnu FAIL
Test 218 cpld_control_nowave_noaero_p8_gnu FAIL
Test 219 cpld_debug_p8_gnu FAIL
Test 220 cpld_control_pdlib_p8_gnu FAIL
Test 221 cpld_debug_pdlib_p8_gnu FAIL

Libraries

  • Not Needed
  • Needed
    • Create separate issue in JCSDA/spack-stack asking for update to library. Include library name, library version.
    • Add issue link from JCSDA/spack-stack following this item
Code Managers Log
  • This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR.
  • Move new/updated input data on RDHPCS Hera and propagate input data changes to all supported systems.
    • N/A

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • Jet
    • Gaea
    • Cheyenne
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@DeniseWorthen DeniseWorthen added bug Something isn't working Baseline Updates Current baselines will be updated. labels Nov 15, 2023
@DeniseWorthen DeniseWorthen marked this pull request as ready for review November 15, 2023 15:32
@DeniseWorthen DeniseWorthen added the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label Nov 16, 2023
@jkbk2004 jkbk2004 mentioned this pull request Nov 17, 2023
42 tasks
@grantfirl
Copy link
Copy Markdown
Collaborator

@DeniseWorthen Since #1993 doesn't change baselines, would you be interested in combining them?

@DeniseWorthen
Copy link
Copy Markdown
Collaborator Author

I would prefer not to combine them since the grv fix changes nearly every baseline. We can of course combine #1993 with any other PR which does not change baselines.

@jkbk2004
Copy link
Copy Markdown
Collaborator

To bring in the bug fix, we can start working on this PR today. @DeniseWorthen @SadeghTabas-NOAA it will be appreciated if WCOSS2/Acorn tests can be covered on your side. @BrianCurtis-NOAA @zach1221 @FernandoAndrade-NOAA FYI

@sadeghitabas
Copy link
Copy Markdown
Contributor

sadeghitabas commented Nov 17, 2023

@jkbk2004 As I talked to @BrianCurtis-NOAA, I don't have sudo permission on wcoss2 and this PR requires BL updates, I'll talk to @junwang-noaa to get the permission by next week.

@zach1221 zach1221 added the jenkins-ci Jenkins CI: ORT build/test on docker container label Nov 17, 2023
@DeniseWorthen
Copy link
Copy Markdown
Collaborator Author

@jkbk2004 I am able to do this PR on WCOSS2.

@epic-cicd-jenkins
Copy link
Copy Markdown
Collaborator

Jenkins-ci ORTs passed

@zach1221
Copy link
Copy Markdown
Collaborator

Ok, looks like testing is wrapped up. Let's proceed with the merging process.

@zach1221 zach1221 merged commit 6a874d1 into ufs-community:develop Nov 20, 2023
@DeniseWorthen DeniseWorthen self-assigned this Feb 22, 2025
@DeniseWorthen DeniseWorthen deleted the bugfix/grvfunction branch November 7, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Baseline Updates Current baselines will be updated. bug Something isn't working jenkins-ci Jenkins CI: ORT build/test on docker container Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

grv function used by NST assumes wrong argument

8 participants