-
Notifications
You must be signed in to change notification settings - Fork 15
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
Updates for ZM round 4 #345
Open
cacraigucar
wants to merge
25
commits into
ESCOMP:development
Choose a base branch
from
cacraigucar:cam_sima_testing
base: development
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
Changes from 24 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
b34d949
Merge remote-tracking branch 'upstream/development' into cam_sima_tes…
cacraigucar d221725
Change to get cam_sima_zm to compile
cacraigucar e4e6b38
Merge remote-tracking branch 'court/history_court' into cam_sima_testing
cacraigucar 7e164e5
Remove nsteps from registry as now in there
cacraigucar b1afafd
Fix typo in metadata file
cacraigucar 03890e8
Merge remote-tracking branch 'court/history_court' into cam_sima_testing
cacraigucar 18a608f
Updates to aid in getting ZM running in CAM-SIMA
cacraigucar ee433ee
Merge remote-tracking branch 'court/history_court' into cam_sima_testing
cacraigucar 3c63ad1
read in all constituents from file; not just build-time ones
28becd1
clarify variable name
75cdffb
fix bug from copied code
006f412
fix unit tests
6bfe67a
Update registry for ZM
cacraigucar dc7b04a
fix air composition standard names; add more water species to registry
3adb55e
Merge remote-tracking branch 'court/read-all-const' into cam_sima_tes…
cacraigucar c12b6cf
Merge remote-tracking branch 'court/read-all-const' into cam_sima_tes…
cacraigucar 30656a2
fix missing line from merge conflict
cacraigucar b93ab05
Merge remote-tracking branch 'upstream/development' into cam_sima_tes…
cacraigucar 77ba066
Merge remote-tracking branch 'upstream/development' into cam_sima_tes…
cacraigucar 6040b85
Merge remote-tracking branch 'upstream/development' into cam_sima_tes…
cacraigucar 85a1096
Update unit test sample files
cacraigucar c33eb21
Fix cloud_area_fraction conflict
cacraigucar 3c5ac5e
Merge tag 'sima0_01_000' into cam_sima_testing
cacraigucar 395fe91
Merge remote-tracking branch 'upstream/development' into cam_sima_tes…
cacraigucar b6c814d
Address reviewer comments
cacraigucar 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -491,5 +491,76 @@ | |
<long_name>graupel mass mixing ratio with respect to moist air plus all airborne condensates</long_name> | ||
<ic_file_input_names>GRAUQM cnst_GRAUQM</ic_file_input_names> | ||
</variable> | ||
|
||
<!-- cam_in Variables --> | ||
<variable local_name="landfrac" | ||
standard_name="land_area_fraction" | ||
units="fraction" type="real" kind="kind_phys" | ||
allocatable="allocatable"> | ||
<dimensions>horizontal_dimension</dimensions> | ||
<ic_file_input_names>landfrac cam_in_landfrac</ic_file_input_names> | ||
</variable> | ||
|
||
<!-- Zhang McFarlane (ZM) Variables --> | ||
<!-- Note that no_deep_pbl defaults to false, but needs to be set to true for diag_TKE or UW schemes --> | ||
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. is this comment outdated? I don't see |
||
<variable local_name="pblh" | ||
standard_name="atmosphere_boundary_layer_thickness" | ||
units="m" type="real" kind="kind_phys" | ||
allocatable="allocatable"> | ||
<dimensions>horizontal_dimension</dimensions> | ||
<ic_file_input_names>pblh pbuf_pblh</ic_file_input_names> | ||
</variable> | ||
<variable local_name="delt" | ||
standard_name="half_timestep_for_physics" | ||
units="s" type="real" kind="kind_phys"> | ||
</variable> | ||
<variable local_name="tpert" | ||
standard_name="convective_temperature_perturbation_due_to_pbl_eddies" | ||
units="K" type="real" kind="kind_phys" | ||
allocatable="allocatable"> | ||
<dimensions>horizontal_dimension</dimensions> | ||
<ic_file_input_names>tpert pbuf_tpert</ic_file_input_names> | ||
</variable> | ||
<variable local_name="ql" | ||
jimmielin marked this conversation as resolved.
Show resolved
Hide resolved
|
||
standard_name="in_cloud_water_vapor_mixing_ratio_wrt_moist_air_and_condensed_water_due_to_deep_convection" | ||
units="kg kg-1" type="real" kind="kind_phys" | ||
allocatable="allocatable"> | ||
<dimensions>horizontal_dimension vertical_layer_dimension</dimensions> | ||
<ic_file_input_names>ICWMRDP pbuf_ICWMRDP</ic_file_input_names> | ||
</variable> | ||
<variable local_name="cldfrc" | ||
standard_name="cloud_area_fraction" | ||
units="fraction" type="real" kind="kind_phys" | ||
allocatable="allocatable"> | ||
<dimensions>horizontal_dimension vertical_layer_dimension</dimensions> | ||
<initial_value>0.7_kind_phys</initial_value> | ||
<ic_file_input_names>CLD pbuf_CLD</ic_file_input_names> | ||
</variable> | ||
<variable local_name="domomtran" | ||
standard_name="flag_for_momentum_transport_by_zhang_mcfarlane_deep_convection_scheme" | ||
units="flag" type="logical" | ||
access="protected" > | ||
<initial_value>.true.</initial_value> | ||
</variable> | ||
<variable local_name="il1g" | ||
standard_name="index_of_first_column_of_gathered_deep_convection_arrays" | ||
units="index" type="integer" | ||
access="protected" > | ||
<initial_value>1</initial_value> | ||
</variable> | ||
<variable local_name="dpdry" | ||
jimmielin marked this conversation as resolved.
Show resolved
Hide resolved
|
||
standard_name="air_pressure_thickness_of_dry_air_for_deep_convection_for_gathered_convective_columns" | ||
units="hPa" type="real" kind="kind_phys" | ||
allocatable="allocatable"> | ||
<dimensions>horizontal_dimension vertical_layer_dimension</dimensions> | ||
</variable> | ||
<variable local_name="fracis" | ||
standard_name="fraction_of_water_insoluble_convectively_transported_species" | ||
units="fraction" type="real" kind="kind_phys" | ||
allocatable="allocatable" | ||
access="protected" > | ||
<dimensions>horizontal_dimension vertical_layer_dimension number_of_ccpp_constituents</dimensions> | ||
<initial_value>1</initial_value> | ||
</variable> | ||
</file> | ||
</registry> |
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 |
---|---|---|
@@ -0,0 +1,24 @@ | ||
|
||
module ppgrid | ||
|
||
!----------------------------------------------------------------------- | ||
! | ||
! Purpose: USED ONLY FOR BACKWARDS COMPATIBILITY WITH CAM!!!! | ||
! PLEASE DELETE ONCE NO LONGER NEEDED BY "to_be_ccppized" | ||
! PHYSICS SCHEMES!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! | ||
! Thanks! | ||
!----------------------------------------------------------------------- | ||
|
||
use physics_grid, only: pcols => columns_on_task | ||
use vert_coord, only: pver => pver, pverp => pverp !WILL NEED TO CHANGE THESE NAMES TO | ||
!SOMETHING ELSE IN CAM-SIMA! | ||
Comment on lines
+13
to
+14
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. i'm guessing this is a question for @nusbaume - do these variable names need to change? |
||
|
||
implicit none | ||
private | ||
save | ||
|
||
public pcols | ||
public pver | ||
public pverp | ||
|
||
end module ppgrid |
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.
can these comments be removed?