Skip to content
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

Generalize computation of moist specific heats #221

Merged
merged 4 commits into from
Sep 20, 2021

Conversation

spencerkclark
Copy link
Member

@spencerkclark spencerkclark commented Sep 13, 2021

This uses the fact that unused tracers will be assigned indexes less than zero to generalize the computation of moist specific heats for more than just the six-species GFDL microphysics scheme. As a result, the warning note below will no longer be emitted when using the Zhao-Carr microphysics scheme:

'GFS_physics_driver::moist_cv - moist_cv for tracer configuration not implemented; using default cv_air for t_dt diagnostics

This change is tested in this notebook. Closes #215.

@spencerkclark spencerkclark marked this pull request as ready for review September 14, 2021 16:20
Copy link
Contributor

@nbren12 nbren12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Awesome that you could implement this in such a general way.

I'm not sure if we need a DSL review for this since. @ofuhrer thoughts?

@ofuhrer
Copy link
Contributor

ofuhrer commented Sep 14, 2021

I'm not sure if we need a DSL review for this since. @ofuhrer thoughts?

I've added @elynnwu as a reviewer for this, since I think she is the right person. In general, DSL is interested in reviewing all PRs in the Fortran code, with highest priority for those that may impact 1) running our configuration 2) numerical results 3) performance.

Copy link
Contributor

@elynnwu elynnwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Some minor comments on naming.

@spencerkclark
Copy link
Member Author

If there are no further comments I will go ahead and merge this at the end of the day.

@spencerkclark spencerkclark merged commit af0b989 into master Sep 20, 2021
@spencerkclark spencerkclark deleted the general-moist-specific-heat branch September 20, 2021 20:48
nbren12 pushed a commit that referenced this pull request Apr 11, 2022
This PR adds files under examples/gmd_timings which run example runfiles used in our upcoming GMD submission and produce timings. Transferred from #221 to give clean changelog.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement moist_cv for the zhao-carr microphysics
4 participants