forked from NCAR/ccpp-physics
-
Notifications
You must be signed in to change notification settings - Fork 54
Tempo version 3 #353
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
Open
AndersJensen-NOAA
wants to merge
39
commits into
ufs-community:ufs/dev
Choose a base branch
from
AndersJensen-NOAA:tempo_v3
base: ufs/dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Tempo version 3 #353
Changes from 32 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
214c182
initial connection of tempo version 3
AndersJensen-NOAA 07b895c
Add comments to radiation for tempo MP and to differentiate tempo fro…
AndersJensen-NOAA 13c04f6
Connects TEMPO to GFS PBL generic
AndersJensen-NOAA fc03db5
fix type and add tempo to aerosol index procedure
AndersJensen-NOAA e29c7fa
simplify tempo driver and add condensation routine
AndersJensen-NOAA d8171f0
radiation to use tempo v2 for now
AndersJensen-NOAA 391e7b3
adds diagnostics and optional arguments to tempo
AndersJensen-NOAA af6f852
add aerosol surface emissions to tempo and aerosol and hail aware flags
AndersJensen-NOAA d6e6ed6
couple tempo to MYNN in ccpp
AndersJensen-NOAA 425e3f0
update pointer to tempo and fix missing qv variable
AndersJensen-NOAA 25bdd3a
update GFS_rrtmg_pre for tempo v3, clean up condensation
AndersJensen-NOAA 2fe1529
update to authoritative repo
AndersJensen-NOAA 4bc4a12
clean-up code and add checks for tempo
AndersJensen-NOAA a5b3fab
clean up radiation coupling to tempo v3
AndersJensen-NOAA ee031b7
update GFS_rad_pre, bugfixes to mp_temp, and update to tempo v3.0.3
AndersJensen-NOAA 0e14599
use thompson functions in interstial 4 for now as tendencies there ap…
AndersJensen-NOAA 8192851
Work to fix interstitial 4
AndersJensen-NOAA d7f451e
update to tempo version 3.0.4
AndersJensen-NOAA a75b7b3
modifications from code review
AndersJensen-NOAA 6be5ef2
add loop for dt_inner on ccpp side
AndersJensen-NOAA 49bff41
add back ability of host to control a set of physical constants used …
6fbe6ab
Merge branch 'ufs/dev' into tempo_v3_gjf
c89a764
fix unit in mp_tempo.meta
cf1f8ff
Merge branch 'ufs/dev' into tempo_v3_gjf
e7acba4
fix compilation error in mp_tempo_condensation.F90 related to odt
a18ccf5
add some more constants to use/only list to compile
f474dbd
roll back units change in mp_tempo.meta to fix weird ccpp_prebuild error
AndersJensen-NOAA cce5984
fix subroutine arguments in fast physics condensation and update temp…
AndersJensen-NOAA d2416f6
Merge branch 'grant_tempo' into tempo_v3
AndersJensen-NOAA a19da35
fixes for RTs
AndersJensen-NOAA 733fff3
update .gitmodules
75b1d5f
fix for ufs-mpas
AndersJensen-NOAA ece309d
revert
AndersJensen-NOAA 97c8dd2
update tempo
AndersJensen-NOAA 08e1bcf
Merge branch 'ufs/dev' into tempo_v3
grantfirl db5a369
update tempo to v3.1.0
AndersJensen-NOAA 2b5603e
Merge branch 'tempo_v3' of https://github.com/AndersJensen-NOAA/ccpp-…
AndersJensen-NOAA 52d5cdb
remove old version of TEMPO that was added back
AndersJensen-NOAA fb004f9
point tempo_v3 submodule to latest top-of-main commit
grantfirl 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 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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,17 +13,18 @@ module GFS_PBL_generic_common | |
|
|
||
| subroutine set_aerosol_tracer_index(imp_physics, imp_physics_wsm6, & | ||
| imp_physics_thompson, ltaerosol,mraerosol, & | ||
| imp_physics_tempo, lthailaware, & | ||
| imp_physics_mg, ntgl, imp_physics_gfdl, & | ||
| imp_physics_nssl, & | ||
| nssl_hail_on, nssl_ccn_on, nssl_3moment, kk, & | ||
| errmsg, errflg) | ||
| implicit none | ||
| ! | ||
| integer, intent(in ) :: imp_physics, imp_physics_wsm6, & | ||
| imp_physics_thompson, & | ||
| imp_physics_thompson, imp_physics_tempo,& | ||
| imp_physics_mg, ntgl, imp_physics_gfdl, & | ||
| imp_physics_nssl | ||
| logical, intent(in ) :: ltaerosol, mraerosol, nssl_hail_on, nssl_ccn_on, nssl_3moment | ||
| logical, intent(in ) :: ltaerosol, mraerosol, nssl_hail_on, nssl_ccn_on, nssl_3moment, lthailaware | ||
| integer, intent(out) :: kk | ||
| character(len=*), intent(out) :: errmsg | ||
| integer, intent(out) :: errflg | ||
|
|
@@ -43,6 +44,18 @@ subroutine set_aerosol_tracer_index(imp_physics, imp_physics_wsm6, & | |
| else | ||
| kk = 9 | ||
| endif | ||
| elseif (imp_physics == imp_physics_tempo) then | ||
| ! Tempo (Note: why are we setting tracer indices manually?) | ||
| ! I'll assume that kk = 9 includes qv, qc, qr, qi, qs, qg, ni, nr | ||
| ! when ltaerosol = true: nc, nwfa, and nifa are added (+3) | ||
| ! when lthail = true: ng and volg are added (+2) | ||
| kk = 9 | ||
| if(ltaerosol) then | ||
| kk = kk + 3 | ||
| endif | ||
| if(lthailaware) then | ||
|
Collaborator
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. mraerosol does not work with TEMPO, is it correct?
Collaborator
Author
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. Correct @mzhangw , it wasn't hooked up to tempo v2 either. We have plans to add this soon! |
||
| kk = kk + 2 | ||
| endif | ||
| ! MG | ||
| elseif (imp_physics == imp_physics_mg) then | ||
| if (ntgl > 0) then | ||
|
|
||
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
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.