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

cpld_gridgen - Update regression tests to run in parallel #1027

Draft
wants to merge 20 commits into
base: develop
Choose a base branch
from

Conversation

GeorgeGayno-NOAA
Copy link
Collaborator

@GeorgeGayno-NOAA GeorgeGayno-NOAA commented Feb 27, 2025

DESCRIPTION OF CHANGES:

Currently, each regression test runs in sequence. These updates submit all tests to the batch queue at once, where they run independently. Then, when they are finished, a 'summary' job is triggered to create some final log files, check for errors and create a "summary" log. All other functions, such as options to build the executable (rt.sh -b), create a new baseline (rt.sh -c) and test against the new baseline (rt.sh -m) are retained.

Some specifics:

  • The check_results function was moved from rt.sh to ./ush/cpld_gridgen.sh
  • New script rt_summary.sh is called from rt.sh. It checks for errors, concatenates the log files from the individual tests to a final log file, checks for errors and creates a summary file.
  • Removed some now unnecessary exports from the scripts.

TESTS CONDUCTED:

If there are changes to the build or source code, the tests below must be conducted. Contact a repository manager if you need assistance.

  • Compile branch on all Tier 1 machines using Intel (Orion, Jet, Hera, Hercules and WCOSS2).
  • Compile branch on Hera using GNU.
  • Compile branch in 'Debug' mode on WCOSS2.
  • Compile with Doxygen on any machine with no errors.
  • Run unit tests locally on any Tier 1 machine.
  • Run relevant consistency tests locally on all Tier 1 machines.

Optional test.

  • Run full set of chgres_cube consistency tests on Hera.

Describe any additional tests performed.

DEPENDENCIES:

None.

DOCUMENTATION:

All new and updated source code must be documented with Doxygen.

  • Doxygen is updated.

If this PR is contributing new capabilities that need to be documented, please also include updates to the RST files in the docs/source directory as supporting material.

ISSUE:

Fixes #1016.

GeorgeGayno-NOAA and others added 20 commits January 22, 2025 16:28
Add the remove of some log files from scripts.

Fixes ufs-community#1016.
@GeorgeGayno-NOAA
Copy link
Collaborator Author

@DeniseWorthen - I would like you to review this. No hurry. The overall functionality is unchanged.

@GeorgeGayno-NOAA
Copy link
Collaborator Author

I still have some final cleanup to do.

@DeniseWorthen
Copy link
Contributor

Oh that's great! I'll take a look when I've got a chance.

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.

Speed up cpld_gridgen tests
2 participants