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

Update CMEPS to its ESCOMP/master branch 20220607 commit #67

Merged
merged 92 commits into from
Jun 9, 2022

Conversation

uturuncoglu
Copy link
Collaborator

Description of changes

ESCOMP/CMEPS master branch has been updated on Jun, 7, need to update emc/develop correspondingly but bring exchange grid capability to UFS Weather model.

Specific notes

Contributors other than yourself, if any:

CMEPS Issues Fixed (include github issue #): NaN

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

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

Testing performed if application target is CESM:

Testing performed if application target is UFS-coupled:

  • (recommended) UFS-coupled testing
    • description: All UFS RTs are passed on Cheyenne
    • details (e.g. failed tests):

Testing performed if application target is UFS-HAFS:

  • (recommended) UFS-HAFS testing
    • description: All UFS RTs are passed on Cheyenne
    • details (e.g. failed tests):

Hashes used for testing:

uturuncoglu and others added 30 commits November 24, 2021 15:39
Exchange grid updates from Dom 2022/01/30
fix aoflux calculation on agrid and add missing error checks
Description of changes

initialize drydep_nflds to 0 in seq_drydep_readnl.

Issue found when running CAM test ERP_Ln9_Vnuopc.ne5pg3_ne5pg3_mg37.QPC6.izumi_nag.cam-outfrq9s_clubbmf with the nuopc driver. Got the following error (and preceding log message):

    [0] (seq_drydep_init) Number of dry deposition fields transfered is **
    ================================================================================
    = BAD TERMINATION OF ONE OF YOUR APPLICATION PROCESSES
    = PID 117892 RUNNING AT i042.cgd.ucar.edu
    = EXIT CODE: 9
    = CLEANING UP REMAINING PROCESSES
    = YOU CAN IGNORE THE BELOW CLEANUP MESSAGES

Specific notes

Contributors other than yourself, if any: N/A

CMEPS Issues Fixed (include github issue #): resolves ESCOMP#272

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

Any User Interface Changes (namelist or namelist defaults changes)? no user interface 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: izumi CAM aux_cam test ERP_Ln9_Vnuopc.ne5pg3_ne5pg3_mg37.QPC6.izumi_nag.cam-outfrq9s_clubbmf with nuopc_driver
        details (e.g. failed tests): No failures.

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:

    repository to check out: ESCOMP/CESM.git
    branch/hash:

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:
@uturuncoglu uturuncoglu changed the title Feature/update Update CMEPS to its ESCOMP/master branch 20220607 commit Jun 7, 2022
@jkbk2004
Copy link

jkbk2004 commented Jun 8, 2022

@MinsukJi-NOAA @BrianCurtis-NOAA can we add one more reviewer? On jet, I am running two more RT cases manually.

@jkbk2004
Copy link

jkbk2004 commented Jun 9, 2022

@binli2337 @junwang-noaa Denise is not available this week. If you have permission on this repo, can you take a look at this PR to merge in? All RTs pass ok on ufs-wm side: #1157

@junwang-noaa junwang-noaa merged commit 424a086 into NOAA-EMC:emc/develop Jun 9, 2022
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.

9 participants