Skip to content

remove the version ifdef and add the CASE_HASH variable#200

Merged
jedwards4b merged 1 commit into
ESCOMP:masterfrom
jedwards4b:remove_esmf_version_ifdef
Jun 4, 2021
Merged

remove the version ifdef and add the CASE_HASH variable#200
jedwards4b merged 1 commit into
ESCOMP:masterfrom
jedwards4b:remove_esmf_version_ifdef

Conversation

@jedwards4b
Copy link
Copy Markdown
Collaborator

Description of changes

Remove the ifdef which assured that the version of ESMF was 8 or greater. Version 8 or greater is always required so this is just a cleanup. Also added the CASE_HASH variable which is used by cime to establish provenance.

Specific notes

Contributors other than yourself, if any:

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers?

  • bit for bit
  • different at roundoff level
  • more substantial

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

  • Yes
  • No

Testing performed if application target is CESM:(either UFS-S2S or CESM testing is required):

  • (recommended) CIME_DRIVER=nuopc scripts_regression_tests.py
    • machines: cheyenne intel
    • 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:

@jedwards4b jedwards4b self-assigned this Jun 2, 2021
Copy link
Copy Markdown
Collaborator

@DeniseWorthen DeniseWorthen left a comment

Choose a reason for hiding this comment

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

Thanks for doing this. EMC was looking to make this same change.

@jedwards4b jedwards4b merged commit 4214425 into ESCOMP:master Jun 4, 2021
@jedwards4b jedwards4b deleted the remove_esmf_version_ifdef branch June 4, 2021 21:05
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.

3 participants