Skip to content

WIP: Fixes for lowres testsuites#413

Merged
rtodling merged 1 commit intodevelopfrom
bugfix/fix-testsuites
Feb 17, 2026
Merged

WIP: Fixes for lowres testsuites#413
rtodling merged 1 commit intodevelopfrom
bugfix/fix-testsuites

Conversation

@mathomp4
Copy link
Member

As found by @weiyuan-jiang, runjob.pl (aka fvsetup) is unhappy with the C90C.input file. The reason was a missing question/answer pair in the file for catchment.

I found something similar in the C48f.input file as well.

NOTE: At the moment, @weiyuan-jiang cannot complete an fvsetup even on the fix. Current issues are:

  1. The C90C.input restarts are missing gwd_internal_rst

@mathomp4 mathomp4 requested a review from rtodling February 17, 2026 15:25
@mathomp4 mathomp4 self-assigned this Feb 17, 2026
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Feb 17, 2026
Copy link
Collaborator

@rtodling rtodling left a comment

Choose a reason for hiding this comment

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

Seems fine by me.

@rtodling rtodling marked this pull request as ready for review February 17, 2026 16:16
@rtodling rtodling requested a review from a team as a code owner February 17, 2026 16:16
@rtodling rtodling merged commit 930847f into develop Feb 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants