Skip to content

Update job checking implementation#554

Merged
apcraig merged 1 commit intoCICE-Consortium:mainfrom
apcraig:derscripts
Feb 9, 2026
Merged

Update job checking implementation#554
apcraig merged 1 commit intoCICE-Consortium:mainfrom
apcraig:derscripts

Conversation

@apcraig
Copy link
Contributor

@apcraig apcraig commented Feb 8, 2026

PR checklist

  • Short (1 sentence) summary of your PR:
    Update job checking implementation
  • Developer(s):
    apcraig
  • Suggest PR reviewers from list in the column to the right.
  • Please copy the PR test results link or provide a summary of testing completed below.
    All tests bit-for-bit, confirmed new implementation of job checking https://github.com/CICE-Consortium/Test-Results/wiki/icepack_by_hash_forks#88477f8ded8b3c4a5b010ce71a081de0ceecaef7
  • How much do the PR code changes differ from the unmodified code?
    • bit for bit
    • different at roundoff level
    • more substantial
  • Does this PR create or have dependencies on CICE or any other models?
    • Yes
    • No
  • Does this PR add any new test cases?
    • Yes
    • No
  • Is the documentation being updated? ("Documentation" includes information on the wiki or in the .rst files from doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.)
    • Yes
    • No, does the documentation need to be updated at a later time?
      • Yes
      • No
  • Please document the changes in detail, including why the changes are made. This will become part of the PR commit log.

Update job checking logic to avoid PBS output that shows completed jobs, added -v " historical ". This is far from ideal and not particularly future proof, but PBS qstat has become a mess.

Fix potential bug in setting ICE_MACHINE_QSTAT if the string has spaces in it.

This is the Icepack fix similar to CICE-Consortium/CICE#1091

added -v " historical ". This is far from ideal and not particularly future proof,
but PBS qstat has become a mess.

Fix potential bug in setting ICE_MACHINE_QSTAT if the string has spaces in it.
@apcraig apcraig merged commit 222ab41 into CICE-Consortium:main Feb 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants