Skip to content

WORK IN PROGRESS: fv3atm changes for splitting physics group in SDF #276

Closed
climbfuji wants to merge 15 commits into
NOAA-EMC:developfrom
climbfuji:split_physics_group_20210409
Closed

WORK IN PROGRESS: fv3atm changes for splitting physics group in SDF #276
climbfuji wants to merge 15 commits into
NOAA-EMC:developfrom
climbfuji:split_physics_group_20210409

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented Apr 9, 2021

Heads-up

Note. Includes all the changes from #261 and #271, because this PR it will be merged afterwards and not having those PRs included in the development phase will lead to merge conflicts.

The commit history and changes will be much smaller once #261 and #271 are merged. The first PR that really belongs to this PR is 861a8c5.

Description

This PR contains the following changes:

  • Introduce GFS_suite_interstitial DDT that contains "persistent interstitial" variables (allocated for each block)
  • Add the necessary logic in ccpp/driver/CCPP_driver.F90 and atmos_model.F90 to call the surface_physics and atmosphere_physics groups instead of the physics group, currently hardcoded for the FV3_GFS_v16 suite
  • Note. further optimization is required to move as many as possible of the suite interstitial DDT components back to the original interstitial DDT to reduce memory footprint

Issue(s) addressed

n/a

Testing

See ufs-community/ufs-weather-model#516

Dependencies

NCAR/ccpp-physics#615
#276
ufs-community/ufs-weather-model#516

…pt RRTMGP interstititals over from GFS_interstitial DDT
…osphere_physics, make corresponding adjustments in ccpp/driver/CCPP_driver.F90 and atmos_model.F90
@climbfuji
Copy link
Copy Markdown
Collaborator Author

Progress has stalled, and the code capabilities doen't seem to be required right now. Closing.

@climbfuji climbfuji closed this Aug 5, 2021
SamuelTrahanNOAA added a commit to SamuelTrahanNOAA/fv3atm that referenced this pull request Sep 29, 2023
LarissaReames pushed a commit to LarissaReames/fv3atm that referenced this pull request Nov 17, 2023
…ength-mismatch

initialize two arrays and fix fortran coding error plus PRs NOAA-EMC#285 and NOAA-EMC#276
zhanglikate pushed a commit to zhanglikate/ufsatm that referenced this pull request Oct 7, 2025
…sics, and string length mismatch in dycore (plus PR #1913, #1917, and #1926) (#1893)

* GFDL_atmos_cubed_sphere: consistent string lengths in array

* stop FV3_HRRR_c3 from crashing with gnu debug

* 1hr forecast limit for conus13km_debug_qr

* fv3atm: bug fix from Dusan to recover_fields crash

* disable conus13km_debug_qr_gnu due to 25% failure rate on Hera

* FV3 dycore: initialize srf_wnd_var2 and tracers_var3 arrays

* enable conus13km_debug_qr_gnu

* Fix race condition in GFS_phys_time_vary.fv3.F90 error detection

* More bug fixes to GFS_phys_time_vary.fv3.F90:
1. detect empty errmsg from subroutines
2. Initialize err variables in set_soilveg.f, which is called from GFS_phys_time_vary.fv3.F90

* ccpp-physics: initialize errmsg & errflg in noahmp_tables.f90

* ccpp-physics: only read h2odata, ozdata and noahmp table when they are needed

* "point to the dimension fix of smc for dust emission"

* FV3: more dycore bug fixes from GFDL_atmos_cubed_sphere PR 285

* merge #1926

* merge GFDL_atmos_cubed_sphere NOAA-EMC#276

* bugfix: 12hr hrrr tests

* add GAEA rocoto support

* fv3: merge GF radar fixes

* check that baseline directory exists and is non-empty

---------

Co-authored-by: Haiqin.Li <Haiqin.Li@noaa.gov>
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.

1 participant