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

Add Izumi version of the aux_clm unit testing #2330

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

samsrabin
Copy link
Collaborator

Description of changes

Because unit testing doesn't work yet on Derecho (see #2329), this PR adds a new test to aux_clm that does unit testing on Izumi.

Specific notes

Contributors other than yourself, if any: None

CTSM Issues Fixed (include github issue #): None

Are answers expected to change (and if so in what way)? No

Any User Interface Changes (namelist or namelist defaults changes)? No

Testing performed, if any: None

@samsrabin samsrabin added the testing additions or changes to tests label Jan 22, 2024
@samsrabin samsrabin self-assigned this Jan 22, 2024
Copy link
Collaborator

@ekluzek ekluzek left a comment

Choose a reason for hiding this comment

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

Perfect. Thank you.

@ekluzek
Copy link
Collaborator

ekluzek commented Jan 24, 2024

Hmmm. @slevis-lmwg we probably haven't run the UNIT tests on izumi in a long time. So we might not have noticed that it's failing. I know it's worked in the past. It might be worth having a couple of us look at this and see if it can be fixed reasonably quickly...

@slevis-lmwg
Copy link
Contributor

Test worked, no problems @ekluzek

    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel CREATE_NEWCASE
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel XML
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel SETUP
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel SHAREDLIB_BUILD time=0
    FAIL FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel NLCOMP
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel MODEL_BUILD time=0
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel SUBMIT
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel RUN time=154
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel GENERATE
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel BASELINE
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel MEMLEAK
    PASS FUNITCTSM_P1x1.f10_f10_mg37.I2000Clm50Sp.izumi_intel SHORT_TERM_ARCHIVER

@slevis-lmwg slevis-lmwg merged commit 2dfd59e into ESCOMP:master Jan 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing additions or changes to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants