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

updates for nersc systems #2147

Merged
merged 1 commit into from
Dec 15, 2017
Merged

Conversation

jedwards4b
Copy link
Contributor

Module updates for cori and edison.

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes
User interface changes?:

Update gh-pages html (Y/N)?:

Code review:

@fischer-ncar fischer-ncar merged commit 39d7442 into ESMCI:master Dec 15, 2017
@jedwards4b jedwards4b deleted the machine_updates branch December 15, 2017 22:31
jgfouca pushed a commit that referenced this pull request Mar 27, 2018
Fix domain files setting for the CONUS RRM configuration. Specification
of the domain files for the CONUS setup were missing from config_grids.xml,
so hgrid was not being set, causing set_horiz_grid to fail during case
setup. Adding specification of which domain files to use fixes this
issue. Fixes #2147.
jgfouca pushed a commit that referenced this pull request Mar 27, 2018
Fix CONUS RRM configuration

Fix two issues with the CONUS RRM configuration that prevented configuring a case with CONUS. First, the domain files were not specified for the CONUS grid in config_grids.xml, so hgrid ended up remaining UNSET, causing set_horiz_grid to fail with all components having grid values UNSET. This PR puts the domain file specification back into config_grids.xml for CONUS, and fixes #2147. Second, an older land initial condition (finidat) file was being used for sim year 2000 runs, which was incompatible with recent changes, causing runs to fail. This PR replaces finidat for sim year 2000 with the updated version.  Fixes #1899

This PR also adds a test suite for RRM grids.

[BFB]
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