Skip to content

Cleanup work for CCPP transition 2019/06/27 - 1#279

Merged
climbfuji merged 1 commit into
NCAR:masterfrom
climbfuji:cleanup_work_gerrit_codereview_20190626
Jun 28, 2019
Merged

Cleanup work for CCPP transition 2019/06/27 - 1#279
climbfuji merged 1 commit into
NCAR:masterfrom
climbfuji:cleanup_work_gerrit_codereview_20190626

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

  • check several issues as described in the notes in the files, remove notes when check is ok
  • remove or reword existing comments, add notes on future cleanup work
  • remove commented code in mp_thompson_post
  • remove hard-coded dimension extents for d?3dt fields to allow running the model without having to allocate the d?3dt fields when ldiag3d is false

… dimension extents for d?3dt fields, add notes on future cleanup work
@climbfuji climbfuji changed the title Cleanup work for CCPP transition Cleanup work for CCPP transition 2019/06/27 - 1 Jun 27, 2019
@climbfuji
Copy link
Copy Markdown
Collaborator Author

@climbfuji
Copy link
Copy Markdown
Collaborator Author

See https://github.com/NCAR/NEMSfv3gfs/pull/188 for regression testing information.

@climbfuji climbfuji marked this pull request as ready for review June 27, 2019 19:21
Copy link
Copy Markdown
Contributor

@llpcarson llpcarson left a comment

Choose a reason for hiding this comment

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

Approved

@climbfuji climbfuji merged commit 8914f4c into NCAR:master Jun 28, 2019
@climbfuji climbfuji deleted the cleanup_work_gerrit_codereview_20190626 branch June 27, 2022 03:09
hannahcbarnes pushed a commit to hannahcbarnes/ccpp-physics that referenced this pull request Aug 3, 2022
…> wat, merra2 threading (NCAR#279)

* changed .gitmodules to point to merra2 ccpp/physics
* remove GFDL_atmos_cubed_sphere and ccpp-framework from .git module
* remove IPD gfsphysics
* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing
* Remove interstitial zorl composites
* Update .gitmodules and submodule pointer fpor ccpp-physics for code review and testing
* Remove or replace references to IPD in comments in atmos_model.F90
* Initialize Sfcprop%zorlx to clear_val instead of huge
* Update submodule pointer for ccpp-physics
* Rename Fortran variables and CCPP standard names / long names of surface composites from ocean to water
* Rename Sfcprop%zorlw to Sfcprop%zorlwav
* Rename Sfcprop%zorlo to Sfcprop%zorlw
* update submodule pointer for ccpp-physics
* Revert change to .gitmodules and update submodule pointer for ccpp-physics
Co-authored-by: anning.cheng <anning.cheng@noaa.gov>
BinLiu-NOAA pushed a commit to hafs-community/ccpp-physics that referenced this pull request Jun 28, 2025
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