Remove nine 3D arrays from CLM Lake model #705
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
There are nine 3D arrays in CLM Lake that are constant. Seven of them are 2D arrays copied to all vertical levels. All are cheap to calculate.
This PR removes all nine arrays. When the numbers are needed, they're recalculated.
To do this, I had to add a 2D array: the lakedepth before correction.
Issue(s) addressed
Testing
How were these changes tested?
A 3km RRFS case was run, removing one array at a time until all nine were gone.
What compilers / HPCs was it tested with?
Hera intel & gnu.
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
Existing tests are sufficient.
Have the ufs-weather-model regression test been run? On what platform?
Hera
Will the code updates change regression test baseline? If yes, why? Please show the baseline directory below.
Yes.
Please commit the regression test log files in your ufs-weather-model branch
Done.
Dependencies