-
Notifications
You must be signed in to change notification settings - Fork 145
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
Add support for Stochastically Perturbed Parameterizations (SPP) in FV3 #820
Merged
Merged
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
b996662
Add SPP option to several physics parameterizations
0947372
bug fixes
a6453ae
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
9fcd6da
checkout dtc/ccpp branch (not main), for rte-rrtmgp
12f92fb
remove optional keyword from metadata files
a851742
specific commit hash for rte-rrtmgp
d5414d2
Add spp args to gwd call, when called directly (vs unified gwd)
e74b9b6
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
f2a2e7d
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
JeffBeck-NOAA 5523ff7
Add missing drag_suite.F90 SPP code
JeffBeck-NOAA c2e8bbf
Requested changes from code review.
JeffBeck-NOAA fbdad7e
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
JeffBeck-NOAA 307a507
Fix dimension-related bug.
JeffBeck-NOAA e44b9aa
Changes for PR modification requests
JeffBeck-NOAA 8d599bc
Fix bug in SPP implementation
JeffBeck-NOAA 6f89824
Perturb cloud* instead of effr*_inout
JeffBeck-NOAA 81d556c
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
JeffBeck-NOAA 9c90b47
Change spp_gwd from logical to integer
JeffBeck-NOAA a7b9875
MYNN SFC perturbation pattern name fix
JeffBeck-NOAA dd419a3
Remove initialization of pattern_spp_sfc=0
JeffBeck-NOAA d466eb1
Updates to standard names and units
JeffBeck-NOAA f685e20
Fix varmax field dimensions
JeffBeck-NOAA 34f79e7
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
JeffBeck-NOAA 73eb0f3
Add dimensions to SPP variables
JeffBeck-NOAA 9650294
Update dimensions of SPP fields
JeffBeck-NOAA 63c5f96
Update dimensions for two SPP fields
JeffBeck-NOAA 9560a79
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
JeffBeck-NOAA 070a9cd
change units of surface_stochastic_weights_from_coupled_process from …
grantfirl 959c7e1
Merge pull request #1 from grantfirl/feature/stoch_spp_gjf
JeffBeck-NOAA 308b0b3
Remove misplaced file.
JeffBeck-NOAA 295cd27
Merge branch 'main' of https://github.com/NCAR/ccpp-physics into feat…
JeffBeck-NOAA def0e78
Update spp_mp to equal 7 in if statement.
JeffBeck-NOAA 372febe
edit module_mp_thompson.F90 to remove optional keyword for rand_pert …
1e02fc8
Merge pull request #2 from grantfirl/feature/stoch_spp_gjf
JeffBeck-NOAA eef1e23
Loop over i,j instead of i,1 for rand_pert field.
JeffBeck-NOAA ac7cde7
Revert dimensions changes to rand_pert.
JeffBeck-NOAA 14ca01e
Revert dimension changes to rand_pert.
JeffBeck-NOAA f07475c
Merge branch 'feature/stoch_spp' of https://github.com/JeffBeck-NOAA/…
JeffBeck-NOAA 88d0dd3
change optional and explicitly-shaped SPP arrays to non-optional and …
grantfirl f77322b
Merge pull request #3 from grantfirl/feature/stoch_spp_gjf
JeffBeck-NOAA File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -624,6 +624,21 @@ | |
dimensions = () | ||
type = logical | ||
intent = in | ||
[spp_wts_gwd] | ||
standard_name = spp_weights_for_gravity_wave_drag_scheme | ||
long_name = spp weights for gravity wave drag scheme | ||
units = 1 | ||
dimensions = (horizontal_loop_extent,vertical_layer_dimension) | ||
type = real | ||
kind = kind_phys | ||
intent = in | ||
[spp_gwd] | ||
standard_name = control_for_gravity_wave_drag_spp_perturbations | ||
long_name = control for gravity wave drag spp perturbations | ||
units = count | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment RE: units. |
||
dimensions = () | ||
type = integer | ||
intent = in | ||
[errmsg] | ||
standard_name = ccpp_error_message | ||
long_name = error message for error handling in CCPP | ||
|
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1257,6 +1257,21 @@ | |
dimensions = () | ||
type = logical | ||
intent = in | ||
[spp_wts_pbl] | ||
standard_name = spp_weights_for_pbl_scheme | ||
long_name = spp weights for pbl scheme | ||
units = 1 | ||
dimensions = (horizontal_loop_extent,vertical_layer_dimension) | ||
type = real | ||
kind = kind_phys | ||
intent = in | ||
[spp_pbl] | ||
standard_name = control_for_pbl_spp_perturbations | ||
long_name = control for pbl spp perturbations | ||
units = count | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment RE: units |
||
dimensions = () | ||
type = integer | ||
intent = in | ||
[lprnt] | ||
standard_name = flag_print | ||
long_name = control flag for diagnostic print out | ||
|
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't the right unit for this, but this can be cleaned up in the future since there are a bunch of wrong units for stuff like this, at least according to https://github.com/ESCOMP/CCPPStandardNames/blob/main/StandardNamesRules.rst
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I'm happy to make those changes in a future PR if it's not urgent right now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not urgent right now and can be merged as-is since we're so far along in the process.