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

dz --> dz_lake bug-fix in LakeTemperatureMod.F90 line 960 #761

Merged
merged 2 commits into from
Jul 15, 2019

Conversation

slevis-lmwg
Copy link
Contributor

@slevis-lmwg slevis-lmwg commented Jul 13, 2019

Description of changes

As described in the title plus a comment correction.

Specific notes

Contributors other than yourself, if any:
@billsacks @dlawrenncar

CTSM Issues Fixed (include github issue #):
fixes #760

Are answers expected to change (and if so in what way)?
Only in diagnostic variable WTGQ

Testing performed, if any:
Cheyenne test suite FAILs with DIFFs from dev049 baseline.
The following commands...
./summarize_cprnc_diffs -basedir /glade/scratch/slevis/tests_0712-173856ch -testid '*'
vi cprnc.summary.*.by_varname
...confirm that only WTGQ is different from dev049.
I will repeat on Hobart.

@slevis-lmwg
Copy link
Contributor Author

slevis-lmwg commented Jul 15, 2019

Ran Hobart's test suite. Then...
./summarize_cprnc_diffs -basedir /scratch/cluster/slevis/tests_0713-111631ho -testid '*'
vi cprnc.summary.*.by_varname
...confirmed that only WTGQ is different from dev049.

@billsacks I generated the baselines for dev050 on cheyenne and hobart while running the above mentioned tests. I will update the /doc directory next.

@billsacks
Copy link
Member

@slevisconsulting thanks a lot for this. It looks to me like this is ready to go. Is tat right? If so, I'll merge it.

@slevis-lmwg
Copy link
Contributor Author

Yes, that's right.

@billsacks billsacks merged commit 1dc59f8 into ESCOMP:master Jul 15, 2019
@slevis-lmwg slevis-lmwg deleted the dz2dz_lake_fix branch July 15, 2019 20:32
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.

dz used instead of dz_lake in LakeTemperatureMod.F90 in a loop with use_lch4 = .true.
2 participants