Skip to content

Convert GFS DDTs from blocked data structures to contiguous arrays: Part Deux + CCPP updates in fv3atm/ccpp-physics: split physics in two groups, reset GFS_interstitial DDT in CCPP_driver.F90 #2651 + Hera move spack-stack to /contrib #2579#2610

Merged
jkbk2004 merged 45 commits into
ufs-community:developfrom
dustinswales:feature/contiguous_arrays
Apr 2, 2025

Conversation

@dustinswales
Copy link
Copy Markdown
Collaborator

@dustinswales dustinswales commented Feb 21, 2025

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
  • Commit 'test_changes.list' from previous step

Description:

Commit Message:

* UFSWM - Hera spack-stack migration to /contrib
  * AQM - 
  * CDEPS - 
  * CICE - 
  * CMEPS - 
  * CMakeModules - 
  * FV3 - Convert from using blocked data structures to contiguous data structures for the GFS external data types (GFS_diagnostics and GFS_restart).
  * FV3 - CCPP updates in fv3atm/ccpp-physics: split physics in two groups, reset GFS_interstitial DDT in CCPP_driver.F90
    * ccpp-physics - Remove GFS_suite_interstitial_{phys,rad}_reset.* (reset in fv3atm CCPP_driver.F90)
    * atmos_cubed_sphere - 
  * GOCART - 
  * HYCOM - 
  * MOM6 - 
  * NOAHMP - 
  * WW3 - 
  * fire_behavior
  * stochastic_physics - 

Priority:

  • Normal.

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Changes.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • None

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • GaeaC5
    • GaeaC6
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

dustinswales and others added 27 commits December 19, 2024 21:05
Also, make more customizable dprefix environmental variable.
@jkbk2004 jkbk2004 changed the title Convert GFS DDTs from blocked data structures to contiguous arrays: Part Deux Convert GFS DDTs from blocked data structures to contiguous arrays: Part Deux + CCPP updates in fv3atm/ccpp-physics: split physics in two groups, reset GFS_interstitial DDT in CCPP_driver.F90 #2651 + Hera move spack-stack to /contrib #2579 Apr 1, 2025
@jkbk2004 jkbk2004 added No Baseline Change No Baseline Change Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. labels Apr 1, 2025
jkbk2004 and others added 10 commits April 1, 2025 07:52
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
 on-behalf-of @ufs-community <ecc.platform@noaa.gov>
@jkbk2004 jkbk2004 removed the jenkins-ort run ORT testing label Apr 2, 2025
@jkbk2004
Copy link
Copy Markdown
Collaborator

jkbk2004 commented Apr 2, 2025

We can start merging process.

@jkbk2004
Copy link
Copy Markdown
Collaborator

jkbk2004 commented Apr 2, 2025

@dustinswales new fv3 hash is NOAA-EMC/ufsatm@fbc09c9

@BrianCurtis-NOAA
Copy link
Copy Markdown
Collaborator

The title says: "Convert GFS DDTs from blocked data structures to contiguous arrays: Part Deux + CCPP updates in fv3atm/ccpp-physics: split physics in two groups, reset GFS_interstitial DDT in CCPP_driver.F90 #2651 + Hera move spack-stack to /contrib #2579"

Yet the PR description does not mention anything about CCPP updates in fv3atm/ccpp-physics: split physics in two groups, reset GFS_interstitial DDT in CCPP_driver.F90 nor Hera move spack-stack to /contrib.

The git tracking does not include any links to the PR's added above. I don't even know if there are issues that those other PR's close that have not been added to this to be closed automatically on merge.

The commit message needs to remove any unused sections.

@jkbk2004 It's been mentioned a lot in the past, but to reiterate due to its importance, please be careful upon merging in PR's together to concatenate all information from all pieces into the one that will be merged. Ideally before final testing.

Prefer quality over quantity.

@jkbk2004
Copy link
Copy Markdown
Collaborator

jkbk2004 commented Apr 2, 2025

@BrianCurtis-NOAA updated the description

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Baseline Change No Baseline Change Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Convert blocked data structures in FV3atm to contiguous arrays

6 participants