Skip to content

Conversation

mark-a-potts
Copy link
Contributor

@mark-a-potts mark-a-potts commented Mar 27, 2025

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:

This PR will enable rt.sh to run using containerized spack-stack version 1.9.1 using mixed intel/llvm compilers and adds a new option to "RUN_ONLY" if the "COMPILE_ONLY" has previously been run.

Commit Message:

* UFSWM - modifies rt.sh and Intel flags used to compile. Adds the option to run tests only (no compile) 
  

Priority:

  • Normal

Git Tracking

UFSWM:

  • Closes #
  • None

Sub component Pull Requests:

  • None

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Changes.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • GaeaC5
    • GaeaC6
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

tests/rt.sh Outdated
fi

create_or_run_compile_task
if [[ ${RUN_ONLY} == true ]]; then
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is something i have wanted to see for a long time.

is this going to enable users to run the ufs-wm 'run' step on an existing fv3.exe to just check if the run will pass (without having to wait for a re-compile)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That is the idea, but it isn't working yet, unfortunately.

@mark-a-potts mark-a-potts force-pushed the feature/containerized-rt branch from 8bc3bc1 to c4cbb71 Compare April 2, 2025 15:22
@mark-a-potts mark-a-potts force-pushed the feature/containerized-rt branch from a6d9ccd to 44a644a Compare April 25, 2025 16:23
@mark-a-potts mark-a-potts force-pushed the feature/containerized-rt branch from 44a644a to adf39bb Compare June 12, 2025 20:08
@gspetro-NOAA
Copy link
Collaborator

@mark-a-potts @ulmononian @EdwardSnyder-NOAA @jkbk2004 Is this PR still planned/desired? I know there has been talk of containerizing the RTs, but it seems like someone else would need to continue the work that Mark started. Could we close this PR an open a new one once it's ready?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: On Pause

Development

Successfully merging this pull request may close these issues.

3 participants