Skip to content

Add sfc flux spec#71

Merged
grantfirl merged 3 commits into
masterfrom
add_sfc_flux_spec
May 31, 2018
Merged

Add sfc flux spec#71
grantfirl merged 3 commits into
masterfrom
add_sfc_flux_spec

Conversation

@grantfirl
Copy link
Copy Markdown
Collaborator

This PR adds functionality to the SCM to use prescribed surface fluxes. A new SDF was added that bypasses the surface schemes and a new "scheme" was added to the schemes in the prebuild script configuration. This PR is in conjunction with PRs for ccpp-physics and gmtb-scm.

@grantfirl grantfirl requested review from climbfuji and llpcarson May 30, 2018 22:19
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji 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 to me. Merging this will cause a conflict for my pending PR #70 but that's ok. When #70 goes in, we will also have to update GFS_init_scm and GFS_finalize_scm: the two modules need to be combined into one and the run routines must become the _init and _finalize routines instead (_run stays empty).

@grantfirl
Copy link
Copy Markdown
Collaborator Author

Thanks, Dom. I would suggest that we wait to merge #71 until #70 is in. I'll make the required changes once #70 is in.

@grantfirl grantfirl merged commit 7e1a995 into master May 31, 2018
@grantfirl grantfirl deleted the add_sfc_flux_spec branch May 31, 2018 23:05
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.

2 participants