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

Update NCEP regtests for Orion: Rocky9 OS #1263

Merged
merged 4 commits into from
Jul 15, 2024

Conversation

MatthewMasarik-NOAA
Copy link
Collaborator

@MatthewMasarik-NOAA MatthewMasarik-NOAA commented Jul 2, 2024

Pull Request Summary

Makes the necessary updates to the NCEP regtests for Orion to account for OS upgrade to Rocky9. This also includes a related bug fix.

Description

Contains the following:

  • Update to module environment for Rocky9 OS (based on current UFS/modulefiles: ufs_common.lua, ufs_orion.intel.lua).
  • New Parmetis built with updated modules (Note: the path remains the same, but a new build is in place).
  • Bug fix associated with a null assignment to variable batchq in matrix_cmake_ncep.
  • Text editor auto cleanup of trailing whitespace.

Please also include the following information:

  • Add any suggestions for a reviewer
  • Mention any labels that should be added:
    • bug, enhancement
  • Are answer changes expected from this PR? Please describe the changes and the reason why in addition to which of the following labels would apply:
    • No change to answers (out_grd, out_pnt), though two tests have files that differ described in the Testing section.

Issue(s) addressed

  • Please list any issues associated with this PR, including those the PR will fix/close. For example:

Commit Message

Updates to NCEP regtests for Orion Rocky9 OS

Check list

Testing

  • How were these changes tested?
    • Two sets of matrix regression tests: develop vs. pr, and pr vs. pr.
  • Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
    • No. Regtests are not needed for HPC center software changes.
  • Have the matrix regression tests been run (if yes, please note HPC and compiler)?
    • Orion / intel.
  • Please indicate the expected changes in the regression test output, (Note the list of known non-identical tests.)
    • develop vs. PR
      • ww3_tp2.1 - netcdf output. An issue will be created to track these differences.
      • ww3_ufs1.1/unstr - only due to added switch, BIN2NC, in netcdf output.
    • PR vs. PR
      • Only known non-b4b's.
  • Please provide the summary output of matrix.comp (matrix.Diff.txt, matrixCompFull.txt and matrixCompSummary.txt):

develop vs. PR branch

**********************************************************************
********************* non-identical cases ****************************
**********************************************************************
mww3_test_03/./work_PR2_UQ_MPI_d2                     (16 files differ)
mww3_test_03/./work_PR2_UNO_MPI_e                     (1 files differ)
mww3_test_03/./work_PR2_UNO_MPI_d2                     (15 files differ)
mww3_test_03/./work_PR1_MPI_e                     (1 files differ)
mww3_test_03/./work_PR3_UQ_MPI_d2                     (16 files differ)
mww3_test_03/./work_PR3_UNO_MPI_e                     (1 files differ)
mww3_test_03/./work_PR3_UNO_MPI_d2_c                     (19 files differ)
mww3_test_03/./work_PR3_UQ_MPI_e                     (1 files differ)
mww3_test_03/./work_PR2_UQ_MPI_e                     (1 files differ)
mww3_test_03/./work_PR3_UQ_MPI_d2_c                     (16 files differ)
mww3_test_03/./work_PR1_MPI_d2                     (26 files differ)
mww3_test_03/./work_PR3_UNO_MPI_e_c                     (1 files differ)
mww3_test_03/./work_PR3_UQ_MPI_e_c                     (1 files differ)
mww3_test_03/./work_PR3_UNO_MPI_d2                     (20 files differ)
mww3_test_09/./work_MPI_ASCII                     (0 files differ)
ww3_tp2.1/./work_PR1                     (1 files differ)
ww3_tp2.1/./work_PR1_MPI                     (1 files differ)
ww3_tp2.1/./work_PR3_UNO                     (1 files differ)
ww3_tp2.1/./work_PR2_UQ                     (1 files differ)
ww3_tp2.1/./work_PR3_UQ                     (1 files differ)
ww3_tp2.1/./work_PR3_UQ_MPI                     (1 files differ)
ww3_tp2.1/./work_PR3_UNO_MPI                     (1 files differ)
ww3_tp2.1/./work_PR2_UNO                     (1 files differ)
ww3_tp2.1/./work_PR2_UQ_MPI                     (1 files differ)
ww3_tp2.1/./work_PR2_UNO_MPI                     (1 files differ)
ww3_tp2.10/./work_MPI_OMPH                     (7 files differ)
ww3_tp2.16/./work_MPI_OMPH                     (4 files differ)
ww3_tp2.6/./work_ST4_ASCII                     (0 files differ)
ww3_ufs1.1/./work_unstr_c                     (1 files differ)
ww3_ufs1.1/./work_unstr_b                     (1 files differ)
ww3_ufs1.1/./work_unstr_a                     (1 files differ)
ww3_ufs1.3/./work_a                     (3 files differ)
 
**********************************************************************
************************ identical cases *****************************
**********************************************************************

PR vs. PR

**********************************************************************
********************* non-identical cases ****************************
**********************************************************************
mww3_test_03/./work_PR2_UQ_MPI_d2                     (16 files differ)
mww3_test_03/./work_PR2_UNO_MPI_e                     (1 files differ)
mww3_test_03/./work_PR2_UNO_MPI_d2                     (15 files differ)
mww3_test_03/./work_PR1_MPI_e                     (1 files differ)
mww3_test_03/./work_PR3_UQ_MPI_d2                     (15 files differ)
mww3_test_03/./work_PR3_UNO_MPI_e                     (1 files differ)
mww3_test_03/./work_PR3_UNO_MPI_d2_c                     (14 files differ)
mww3_test_03/./work_PR2_UQ_MPI_e                     (1 files differ)
mww3_test_03/./work_PR3_UQ_MPI_d2_c                     (15 files differ)
mww3_test_03/./work_PR1_MPI_d2                     (15 files differ)
mww3_test_03/./work_PR3_UNO_MPI_e_c                     (1 files differ)
mww3_test_03/./work_PR3_UQ_MPI_e_c                     (1 files differ)
mww3_test_03/./work_PR3_UNO_MPI_d2                     (12 files differ)
mww3_test_09/./work_MPI_ASCII                     (0 files differ)
ww3_tp2.10/./work_MPI_OMPH                     (7 files differ)
ww3_tp2.16/./work_MPI_OMPH                     (4 files differ)
ww3_tp2.6/./work_ST4_ASCII                     (0 files differ)
ww3_ufs1.3/./work_a                     (3 files differ)
 
**********************************************************************
************************ identical cases *****************************
**********************************************************************

@MatthewMasarik-NOAA MatthewMasarik-NOAA marked this pull request as ready for review July 9, 2024 18:32
@MatthewMasarik-NOAA
Copy link
Collaborator Author

Ready for review now, @JessicaMeixner-NOAA

@JessicaMeixner-NOAA
Copy link
Collaborator

@MatthewMasarik-NOAA thanks - I think you correctly pointed out some really strange behavior for tp2.1. Let's open an issue for that. I'll run the regtests one more time and then we can merge this in but I think it's likely good to have the tp2.1 differences well documented because I don't think what's happening now is correct.

@MatthewMasarik-NOAA
Copy link
Collaborator Author

Issue #1267

Copy link
Collaborator

@JessicaMeixner-NOAA JessicaMeixner-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I obtained the same results as @MatthewMasarik-NOAA

Issue #1267 is a concern that should be further investigated particularly as the answers on orion are different than what was previously gotten on orion and what we get on hera. However, this will enable us to otherwise use orion, so we can merge.

@JessicaMeixner-NOAA JessicaMeixner-NOAA merged commit ca58c13 into NOAA-EMC:develop Jul 15, 2024
22 checks passed
@MatthewMasarik-NOAA MatthewMasarik-NOAA deleted the rocky9 branch July 16, 2024 20:22
ukmo-ccbunney added a commit to ukmo-waves/WW3 that referenced this pull request Sep 4, 2024
* origin/develop:
  Enable doxygen documentation in the cmake build system (NOAA-EMC#1281)
  Simplify MPI ifdefs in subroutine W3MPIO (NOAA-EMC#1266)
  Add depth scaling value to SMC regression tests. (NOAA-EMC#1264)
  Updates to NCEP regtests for Orion Rocky9 OS(NOAA-EMC#1263)
  Fix code stability issue in ww3_outp (NOAA-EMC#1258)
  Fix GNU regtest CI failure (NOAA-EMC#1253)
ukmo-ccbunney added a commit to ukmo-waves/WW3 that referenced this pull request Sep 4, 2024
* origin/develop:
  Enable doxygen documentation in the cmake build system (NOAA-EMC#1281)
  Simplify MPI ifdefs in subroutine W3MPIO (NOAA-EMC#1266)
  Add depth scaling value to SMC regression tests. (NOAA-EMC#1264)
  Updates to NCEP regtests for Orion Rocky9 OS(NOAA-EMC#1263)
  Fix code stability issue in ww3_outp (NOAA-EMC#1258)
  Fix GNU regtest CI failure (NOAA-EMC#1253)
ukmo-ccbunney added a commit to ukmo-waves/WW3 that referenced this pull request Sep 4, 2024
* feature/gpu/w3srce_refactor:
  Enable doxygen documentation in the cmake build system (NOAA-EMC#1281)
  Simplify MPI ifdefs in subroutine W3MPIO (NOAA-EMC#1266)
  Add depth scaling value to SMC regression tests. (NOAA-EMC#1264)
  Updates to NCEP regtests for Orion Rocky9 OS(NOAA-EMC#1263)
  Fix code stability issue in ww3_outp (NOAA-EMC#1258)
  Fix GNU regtest CI failure (NOAA-EMC#1253)
  Add option to use NetCDF output instead of binary for point output (NOAA-EMC#1230)
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

Successfully merging this pull request may close these issues.

Update/fix NCEP regtests for Rocky9 OS on Orion
2 participants