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

fixes for auxhist output #317

Merged
merged 5 commits into from
Nov 2, 2022
Merged

Conversation

mvertens
Copy link
Collaborator

Description of changes

fixes for auxhist output

Specific notes

These are fixes to get the auxiliary output working for cesm spinup runs.

Contributors other than yourself, if any:

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers? (specify if bfb, different at roundoff, more substantial)

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

Testing performed

Testing performed if application target is CESM:

  • (recommended) CIME_DRIVER=nuopc scripts_regression_tests.py
    • machines:
    • details (e.g. failed tests):
  • (recommended) CESM testlist_drv.xml
    • machines and compilers:
    • details (e.g. failed tests):
  • (optional) CESM prealpha test
    • machines and compilers
    • details (e.g. failed tests):
  • (other) please described in detail
    • machines and compilers
    • details (e.g. failed tests):

Testing performed if application target is UFS-coupled:

  • (recommended) UFS-coupled testing
    • description:
    • details (e.g. failed tests):

Testing performed if application target is UFS-HAFS:

  • (recommended) UFS-HAFS testing
    • description:
    • details (e.g. failed tests):

Hashes used for testing:

  • CESM:
  • UFS-coupled, then umbrella repostiory to check out and associated hash:
    • repository to check out:
    • branch/hash:
  • UFS-HAFS, then umbrella repostiory to check out and associated hash:
    • repository to check out:
    • branch/hash:

@mvertens mvertens marked this pull request as draft October 26, 2022 19:51
@mvertens
Copy link
Collaborator Author

@fischer- could you please use this branch in alpha10c and try to see if the prealpha tests are bfb?
I'm running out of time to get all of these changes in.

@mvertens mvertens marked this pull request as ready for review November 2, 2022 01:48
@DeniseWorthen
Copy link
Collaborator

I tested in UFS and everything passed.

@mvertens
Copy link
Collaborator Author

mvertens commented Nov 2, 2022

@denise, @uturuncoglu, @jedwards4b - thanks so much for your quick response to this PR.

@jedwards4b jedwards4b merged commit fa2ac92 into ESCOMP:master Nov 2, 2022
@mvertens mvertens deleted the feature/fix_auxhist branch June 16, 2023 08:07
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.

4 participants