CCPP Framework Meeting Minutes 2024 04 29

Agenda

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

---

Attendees: Mike Kavulich,

GitHub issues/PRs

CCPP Framework (issues, PRs, discussions)

• PR 549: Constituent updates
  • Waiting for re-review from Steve, Michael
• Issue 553: Rethink CCPP framework development cycle and branch structure
  • Still needs discussion/decision
• PR 555: Cleanup, adding unit tests for ccpp_track_variables.py, activate "chunked data" CI test, add missing MPI target in src/CMakeLists.txt (includes now-closed #556)
  • Approved, waiting for testing
• PR 558: Add script to check fortran vs metadata without a host model.
  • Waiting for re-review from Steve

Standard names (issues, PRs, discussions)

• Issue 57: Feedback on the meaning of: sea_ice_area_fraction_of_sea_area_fraction
  • I still need to update this with latest discoveries about inconsistent use
• Issue 63: Dealing with unusual units
  • Updated with more questions; found some points in CF conventions that need clarification
• PR 67: Use of plural for _in_atmosphere_layer
  • This might be against CF conventions
• PR 68: Need a versioning system

New items for discussion

Updates from last time

Previous notes

Other business

Meeting notes

CCPP Framework

• Michael K added to development cycle issue
• Michael W discussion on using external library for fortran parsing
  • People to chime in asynchronously
  • Michael W - would prefer to make sure it could work before asking people to install things for us
• New issues
  • #557 - python 3.12 bug with string representation of OrderedDict
    • Courtney to change the tests to return keys/values to compare to instead of string representation of OrderedDict
  • #559 - runtime-only variables not being promoted/allocated correctly
    • Courtney has fix and will issue a PR when more fully tested
  • #561 - multi-instance in framework
    • unclear how this connects to the navy's recent scheme solution (Dom to look into)

Standard Names

• Issue #63 - dealing with unusual units
  • Jesse - natural log of air pressure at interfaces/midpoints (not a vertical coordinate)
  • Michael K to proceed with original proposition ("other")
• Issue #67 - use plural for _in_atmosphere_layer
  • Michael K recommended sticking with CF conventions and using _in_atmosphere_layer (no plural)
    • Jesse is convinced
    • Cheryl - does this mean we should revisit at_interfaces?
      • Jesse - “at_interfaces” parallels “at_top_interfaces”, which is necessary to distinguish from “at_top_interface”

Discussion

• Cheryl would like to revisit Issue 553: Rethink CCPP framework development cycle and branch structure in a coming meeting (when we have broader representation)
  • Michael K to bring up in DTC meeting
  • SIMA SEs to discuss Michael K's proposal internally as well