CCPP Framework Meeting Minutes 2025 12 11

Agenda

• GitHub issues/PRs
• New items
• Updates
• Other business
• notes

---

Attendees: Michael Kavulich, Dom Heinzeller, Courtney Peverley, Jesse Nusbaumer, RhaeSung Kim, Cheryl Craig, Dustin Swales

GitHub issues, PRs, discussions

PRs

• Add diagnostic metadata property to constituents object; use existing diagnostic name metadata attribute#695
  • Needs Courtney to address Steve's comment
• Change CODEOWNERS to use teams rather than individuals #701
  • Need discussion on the specifics of Teams
• For discussion: initial configuration of Codee Fortran formatter with examples (src/*.F90) #707

Issues

• Implement abstract type for NetCDF I/O Issue#697
• Add flexible/extensible constituent properties functionality Issue#698
• Handle xmllint versions that return zero even if validation fails Issue#700
• Adopt a Python formatter and linter for the framework Issue#702
• Adopt a Fortran formatter and static code analyzer for CCPP framework Issue#703
• Modify nested suite expansion to preserve comments Issue#704
• Freeze main branch, update tagging naming conventions and wiki (development workflow) Issue#705
  • Everything is done, just need to make sure everyone has had a chance to review the changes to the development guidelines

Drafts/On Hold

• New test to demonstrate GNU failure #689
  • Associated issue: Unallocated fields passed into Group Caps #690
  • PR Demonstrates test failure, but doesn't fix it yet

Capgen project board

• No new issues

New items for discussion

Updates from last time

Previous notes

Other business

Meeting notes

Pull requests

• Diagnostic property (#695)
  • Needs Courtney to respond to review requests
• Change CODEOWNERS to use teams rather than individuals (#701)
  • Michael created the teams
    • Will create NorESM group for Steve
  • Will add branch protection to require 3 reviews for merging to develop
• For discussion: initial configuration of Codee Fortran formatter with examples (#707)
  • .codee-format file gets applied to every subdirectory
  • Moving from 3 spaces to 2 spaces for indentation
  • People to take a look at the settings & fortran mods (easiest to review if you “hide whitespace”)
  • Dom will implement a further 2 PRs:
    • PR to get codee into github actions
    • PR to integrate codee into generated fortran code

Issues

• Implement abstract type for NetCDF I/O (#697)
  • Allows CAM-SIMA to use pio, but other host models to use other libraries
  • Could implement other reader types (like hdf5)
  • Related to https://github.com/NCAR/ccpp-physics/issues/1106
    • mpi_utils.F90 comes with ccpp-physics
    • Might be able to combine with netCDF reader object
    • Dom can create this PR so we’re all looking at the same thing
• Add flexible/extensible constituent property functionality (#698)
  • People to add their thoughts as they have them
  • Would need to standardize the names of these custom properties
• Handle xmllint versions that return zero even if validation fails (#700)
  • Some bad versions of xmllint
  • Need to update to check the stdout and stderr as well as the code
    • Steve has a fix (he’s the only one with a bad version)
    • Will be potentially tricky to test going forward
  • Could alternatively wrap xmllint
• Adopt a python formatter and linter for the framework (#702)
  • Dom reached out to spack developers - they recommended ruff
• Adopt a fortran formatter and static code analyzer for CCPP framework (#703)
  • Related to PR discussed above
• Modify nested suite expansion to preserve comments (#704)
  • Courtney has a branch with the mods, can make a PR when she gets time
• Freeze main branch, update tagging naming conventions and wiki (#705)
  • Only thing left is to document all of this
  • Michael updated development wiki page; everyone approves of the changes

Will meet next week.