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

Update master from gsd/develop 2020/11/20 #338

Merged

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Nov 21, 2020

This PR updates ccpp_prebuild.py to handle array variables that are independent of the blocking structure in the _init and _finalize calls. This is required to perform the physics initialization properly.

The restrictions are that such variables that do not contain a horizontal dimension cannot depend on the block number - this makes sense, because it would be a waste of memory.

The test and use case for this block of code is the RUC LSM initialization, which has been verified to work correctly in the gsd/develop branch for several months now.

For regression testing, see ufs-community/ufs-weather-model#297

Associated PRs:

#338
NCAR/ccpp-physics#521
NOAA-EMC/fv3atm#202
ufs-community/ufs-weather-model#297

climbfuji and others added 14 commits July 7, 2020 11:07
…elop_20200630

Update master from gsd/develop 2020/06/30
…_master_20200707

Update gsd/develop from NCAR master 2020/07/07
…_master

Update gsd/develop from NCAR master
…er_20200903

Update gsd/develop from master 2020/10/01
…without horizontal dimension not supported at this time
…in_init_and_finalize

Support "unblocked" variables in init and finalize
…ter_20201118

Update gsd/develop from master 2020/11/18
Copy link

@JulieSchramm JulieSchramm 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.

Copy link
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

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

Looks OK to me.

@climbfuji climbfuji merged commit f1dc8d6 into NCAR:master Nov 24, 2020
@climbfuji climbfuji deleted the update_master_from_gsd_develop_20201120 branch June 27, 2022 03:10
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.

5 participants