Updated land perturbation scheme#28
Merged
Merged
Conversation
… variables are in generic (i.e., variable agnostic) arrays
pjpegion
approved these changes
Aug 21, 2020
Collaborator
pjpegion
left a comment
There was a problem hiding this comment.
Changes here look good
pjpegion
approved these changes
Aug 26, 2020
climbfuji
approved these changes
Aug 26, 2020
Collaborator
climbfuji
left a comment
There was a problem hiding this comment.
Thanks for making these changes, this looks good to me!
Collaborator
Author
|
I just sent you an email - I pushed it, then realised it's not compiling.
…On Wed, Aug 26, 2020 at 4:27 PM Dom Heinzeller ***@***.***> wrote:
***@***.**** approved this pull request.
Thanks for making these changes, this looks good to me!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#28 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AH7BYD544R6263OMZJMH243SCWD3RANCNFSM4PJISWCQ>
.
--
--------------------------------------------------------------------
Clara Draper
CIRES / NOAA Earth System Research Laboratories
Physical Sciences Laboratory
325 Broadway, Boulder, CO
DSRC, Rm 1D-105
+1 303 497 4467
--------------------------------------------------------------------
|
pjpegion
approved these changes
Aug 31, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Clean up of the pre-existing land perturbation scheme (was called with do_sfcperts) that was intended for use with GEFS forecasts. This scheme is not being used, and needed some clean-up to work robustly. This scheme has been retained as lndp_type=1 (output is zero diff to old scheme) – think about removing in the future?
Addition of new land surface perturbation option (lndp_type=2), intended for use in cycling DA, and designed to sit outside of the forecast model code.
Regressions tests have passed on hera, and I’ve added a fv3_ccpp_lndp regression test for the lndp_type=2 option.
Detailed list of changes:
In stochastic physics:
*For lndp_type=2, added capacity to apply land perturbations throughout forecast (do_advance_pattern), and to include the land perturbations in the stochini file.
Also, fixed a couple of minor bugs, most notably:
*added missing error trapping in stochastic physics (may not have done this everywhere that is necessary).
In FV3:
*merged run_stochastic_physics_sfc routine into run_stochastic_physics
In ccpp/physics:
*variable name updates
In UFS:
• Updated variable names in namelists
• Addition of lndp test
ASSOCIATED PULL REQUESTS:
stochastic_physics 28
ccpp-physics 480
fv3atm 148
ufs-weather-model 176