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

Revert default aoflux_grid to ogrid #344

Merged
merged 1 commit into from
Mar 16, 2023
Merged

Conversation

billsacks
Copy link
Member

Description of changes

xgrid was causing restart problems; revert this until we can solve those problems

Specific notes

Contributors other than yourself, if any: Testing from @fischer-ncar , consultation with @jedwards4b

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers? (specify if bfb, different at roundoff, more substantial) Yes - answer changes for cases that compute atm-ocn fluxes

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

Testing performed

**No testing done exactly on this branch, but @fischer-ncar did testing with both cmeps0.14.16 and cmeps0.14.14 (note that cmeps0.14.14 was the tag that appeared before cmeps0.14.16 due to an oddity in tag ordering); this tag is a blend of those two.

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:

xgrid was causing restart problems; revert this until we can solve those
problems
@jedwards4b jedwards4b merged commit 24ac1ba into ESCOMP:main Mar 16, 2023
@billsacks billsacks deleted the revert_xgrid branch March 16, 2023 14:46
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.

2 participants