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

Correct ocn_co2_type setting for bcrd case #3294

Merged
merged 1 commit into from
Nov 7, 2019

Conversation

jonbob
Copy link
Contributor

@jonbob jonbob commented Nov 7, 2019

This PR brings in a bug fix to correct the ocn_co2_type setting for bcrd cases in the maint-1.1 branch

Fixes #3289

[BFB]

@jonbob jonbob added mpas-ocean bug fix PR BFB PR leaves answers BFB maint-1.1 PRs for or issues about maint-1.1 BGC labels Nov 7, 2019
@jonbob jonbob requested a review from maltrud November 7, 2019 15:54
@jonbob jonbob self-assigned this Nov 7, 2019
jonbob added a commit that referenced this pull request Nov 7, 2019
Correct ocn_co2_type setting for bcrd case

This PR brings in a bug fix to correct the ocn_co2_type setting for bcrd
cases in the maint-1.1 branch

Fixes #3289

[BFB]
@jonbob jonbob merged commit fadb019 into maint-1.1 Nov 7, 2019
@jonbob
Copy link
Contributor Author

jonbob commented Nov 7, 2019

merged to maint-1.1

@jonbob jonbob deleted the jonbob/maint-1.1/fix-ocean-bcrd branch November 7, 2019 16:01
@rljacob
Copy link
Member

rljacob commented Nov 26, 2019

@jonbob @kevans32 does this bug fix change answers for the BCRD cases?

@jonbob
Copy link
Contributor Author

jonbob commented Nov 26, 2019

@rljacob - it does change answers for tests run out-of-the-box. My understanding is that this change was implemented in the run_e3sm script but also committed to the repo to make sure it didn't get lost.

rljacob pushed a commit that referenced this pull request Apr 21, 2021
These variables should be initialized even if for some reason the namelist isn't read.

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: [bit for bit, roundoff, climate changing]

Fixes (See issue) #3294

User interface changes?:

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

Code review:
rljacob pushed a commit that referenced this pull request Apr 21, 2021
These variables should get reasonable initial values if the namelist isn't read.

Test suite: scripts_regression_tests.py had testing of SMS_Ld5.f09_g17.B1850G.cheyenne_intel.allactive-cism-test_coupling
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes #3294

User interface changes?:

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

Code review:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB BGC bug fix PR maint-1.1 PRs for or issues about maint-1.1 mpas-ocean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants