-
Notifications
You must be signed in to change notification settings - Fork 133
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
CCPP'ize dadadj #1026
base: cam_development
Are you sure you want to change the base?
CCPP'ize dadadj #1026
Conversation
… to redirect this to the tagged version
… name is in Filepath
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.
Looks good! Just had a couple of concerns related to the new error message generation.
Externals_CAM.cfg
Outdated
@@ -50,9 +50,9 @@ tag = ALI_ARMS_v1.0.1 | |||
required = True | |||
|
|||
[atmos_phys] | |||
tag = atmos_phys0_02_006 | |||
branch = dadadj |
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.
Just adding the usual reminder to point to a tag once ESCOMP/atmospheric_physics#91 has been approved and merged.
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. Will do. Nice to know the order these things need to be done.
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.
I found it easier to review this along with the atmospheric_physics PR.
call outfld('DADADJ_PD', dadpdf(:ncol,:), ncol, lchnk) | ||
|
||
! convert the t tendency to an s tendency for cam | ||
ptend%s(:ncol,:) = ptend%s(:ncol,:) * cpairv(:ncol,:,lchnk) |
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.
@nusbaume - Just wanting to confirm our definition of tendency_of_air_temperature
? Is it without out the cpairv adjustment? If so, we may want to augment the conversion document to indicate the ptend%s special adjustment?
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.
Yes, tendency_of_air_temperature
is basically the change in state%t
(or air temperature), while ptend%s
is technically the tendency_of_dry_air_enthalpy_at_constant_pressure
, which in CAM we use to calculate the change state%t
in physics_update
by dividing by cpairv
. So in other words:
ptend%t = ptend%s / cpairv
or in standard name lingo:
tendency_of_air_temperature = tendency_of_dry_air_enthalpy_at_constant_pressure / composition_dependent_specific_heat_of_dry_air_at_constant_pressure
Hopefully that makes sense? If not just let me know. Regardless I can say that I am pretty sure this new CAM code is correct as-is.
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.
Everything looks great now, thanks!
Refactor dadadj parameterization for inclusion in the CCPP framework. On the CAM side this entails moving the core parameterization over to atmos_phys and updating the dadadj_cam interface file to call the new library routine. There are additional mods to cime and ccs_config that are needed for the CAM-SIMA dadadj CCPP validation.