ctsm5.4.043 Overflow respiration bug fixes #3491

[wwieder]

Description of changes

Fixes overflow respiration error described in #3491

Specific notes

Contributors other than yourself, if any:

• @katierocci , @slevis-lmwg

CTSM issues resolved or otherwise addressed, if any:
Resolves #3491

If answers are expected to change, describe (delete this line otherwise):
Yes, reduces N limitation in MIMICS simulations

Any user interface changes (namelist or namelist defaults changes)?
NA

Testing planned or performed, if any:

• Single point cases ran at East River and Cedar Creek LTER look improved

Requirements before merge:

• I have followed the CTSM contribution guidelines.

• The code in this PR branch builds with no errors.

• The code in this PR branch runs with no errors. Briefly describe tested configuration(s):

• This either (a) does not change answers, (b) it only changes answers at roundoff level, or (c) I have performed a scientific evaluation of the answer changes. Which?:

  • c, evaluated results in single point cases

• I have reviewed relevant parts of the CLM documentation Tech Note or User's Guide to determine if anything needs to be changed or added. If it does, describe:

  Documentation for MIMICS still needed, see Docs needed: MIMICS #3166

• This PR either (a) does not create a need to update the documentation or (b) includes required documentation updates (see guidelines for contributing documentation). Which?:

  Documentation for MIMICS still needed, see Docs needed: MIMICS #3166

[wwieder]

Minor change, do we need to request a review from someone else?

[slevis-lmwg]

• aux_clm on derecho when the machine comes up: ./run_sys_tests -s aux_clm --skip-generate -c ctsm5.4.038
  Expected diffs:

FAIL ERP_D_P64x2_Ld3.f10_f10_mg37.I1850Clm60BgcCrop.derecho_gnu.clm-mimics BASELINE ctsm5.4.038: DIFF
FAIL SMS_D.1x1_brazil.I1850Clm60BgcCrop.derecho_gnu.clm-mimics_matrixcn BASELINE ctsm5.4.038: DIFF
FAIL SMS_Ld5_Mmpi-serial.1x1_brazil.IHistClm60Bgc.derecho_gnu.clm-mimics BASELINE ctsm5.4.038: DIFF
FAIL SMS_Ly5_Mmpi-serial.1x1_brazil.IHistClm50BgcQianRs.derecho_intel.clm-newton_krylov_spinup BASELINE ctsm5.4.038: DIFF

[slevis-lmwg]

@wwieder do you want to pick another reviewer? Maybe Sam R. would be a good option?

[wwieder]

@wwieder do you want to pick another reviewer? Maybe Sam R. would be a good option?

I think this makes sense. Should we see what derecho testing look like first, or just go ahead and request Sam's review?

[slevis-lmwg]

@wwieder do you want to pick another reviewer? Maybe Sam R. would be a good option?

I think this makes sense. Should we see what derecho testing look like first, or just go ahead and request Sam's review?

I say let's wait, just in case derecho testing discovers problems that require code revisions.

[wwieder]

@slevis-lmwg I think we can merge this PR.

[slevis-lmwg]

./run_sys_tests -s aux_clm -c ctsm5.4.042 -g ctsm5.4.043

• on izumi, and I updated read permissions in the baseline directory
• on derecho
  FAIL ERP_P64x2_D_Ld5.f10_f10_mg37.I2000Clm50Sp.derecho_gnu.clm-default--clm-nofireemis MEMLEAK memleak detected, memory went from 1039.200000 to 1168.030000 in 0 days
  Haven't seen this one before. Opened issue MEMLEAK fail in ERP_P64x2_D_Ld5.f10_f10_mg37.I2000Clm50Sp.derecho_gnu.clm-default--clm-nofireemis #4077 rather than delaying the merge. The test passed on resubmit.

Also this test is on the third resubmit (expected failure that goes away):
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso--clm-matrixcnOn_ignore_warnings

[slevis-lmwg]

• Update ChangeLog/Sum

[slevis-lmwg]

• Ready to merge, but I will wait for feedback from the group regarding the MEMLEAK failure