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

Add lost code to set variable based on ic_mode, for ICG compsets. #1297

Merged
merged 1 commit into from
Mar 2, 2017

Conversation

jonbob
Copy link
Contributor

@jonbob jonbob commented Mar 2, 2017

This PR will add back some coding to set the mpas-cice config_initial_condition_type based on the IC_MODE in build-namelist. This is required for ICG compsets and was somehow lost.

[BFB]

jonbob added a commit that referenced this pull request Mar 2, 2017
Add lost code to set variable based on ic_mode, for ICG compsets

This PR will add back some coding to set the mpas-cice
config_initial_condition_type based on the IC_MODE in build-namelist. This is
required for ICG compsets and was somehow lost.

[BFB]
@jonbob jonbob merged commit 7ebad35 into master Mar 2, 2017
jonbob added a commit that referenced this pull request Mar 2, 2017
Add lost code to set variable based on ic_mode, for ICG compsets

This PR will add back some coding to set the mpas-cice
config_initial_condition_type based on the IC_MODE in build-namelist. This is
required for ICG compsets and was somehow lost.

[BFB]
@jonbob jonbob deleted the jonbob/cice/fix_ic_mode_usage branch July 25, 2017 17:11
jgfouca pushed a commit that referenced this pull request Sep 15, 2017
Get PFUnit working on melvin.
Fix up serial settings in melvin.
Need to explicitly declare PFUNIT_PATH in cmake calls in run_tests.py
Need to load env before making env XML files, some compiler vars pull from environment
Need to not crash when CMAKE_Fortran_COMPILER_ID is empty
Test suite: scripts_regression_tests.py N_TestUnitTest
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #1297

User interface changes?: N

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

Code review: @billsacks @jedwards4b (confirmed N_TestUnitTest still works on cheyenne)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants