-
Notifications
You must be signed in to change notification settings - Fork 64
CCPP Framework Meeting Minutes 2024 01 08
Grant Firl edited this page Jan 8, 2024
·
11 revisions
- GitHub issues/PRs
- New items
- Updates
-
Other business
- [notes] Michael W. and Courtney to be added as CODEOWNERS to ccpp-framework
Attendees: Mike Kavulich, Grant Firl, Courtney Peverley, Ligia Bernardet, Jesse Nusbaumer, Lulin Xue, Michael Waxmonsky, Jordan Powers, Matt Dawson
CCPP Framework (issues, PRs, discussions)
-
PR 504: Feature capgen transforms
- Ready to merge?
-
PR 512: Add debug switch to capgen (perform variable allocation checks etc)
- Ready to merge?
-
PR 517: CCPP-physics Issue #127 Fix
- Apologies for slow merge, did not realize this wasn't done before Christmas break
- will merge after meeting
-
PR 519:
- Needs one more review - Grant TODO
- Associated issue; Issue 520: ccpp_prebuild.py: add capability to use chunks of contiguous arrays in the run phase
-
Issue 521: Add check to disallow usage of horizontal_dimension in the run phase
- Necessary for capgen and prebuild?
- Courtney to discuss with Dustin and Dom
Standard names (issues, PRs, discussions)
-
Issue 54: Need to apply text-to-link conversion rules in Markdown generation
- Associated PR: PR 55: Add function to apply conversion rules to generated markdown links
- Needs another approval before merge.
- Associated PR: PR 55: Add function to apply conversion rules to generated markdown links
-
PR 56: Add new standard names from NCAR idealized physics schemes
- Needs more discussion
- Issue 57: Feedback on the meaning of: sea_ice_area_fraction_of_sea_area_fraction
- Does the init-phase of the CCPP assume that model grid information is (or is not) available at that time?
- This is not explicitly assumed, although SCM and FV3 physics_init phases are called after the grid is available.
- It would be good to make this explicit in the docs for host models.
- Can we introduce a new CCPP phase that can be used before the grid is known? (asked by Jesse N. - will start an issue)
- Timelines for PR reviews
- Email reminders from Repo manager?
-
Discussion 518: Setting error codes other than 1?
- Jesse N: can also use standard IOSTAT codes (don't re-use codes that are already Fortran standard)
- Cheryl C: errmsg can also be used to differentiate when there are repeats (i.e. this error was from an ALLOCATE statement, READ, etc.)
- Example for Intel: https://www.cis.uoguelph.ca/~wgardner/intel_docs/compilers/compiler_f/main_for/bldaps_for/common/bldaps_rterrs.htm
- https://www.intel.com/content/www/us/en/docs/fortran-compiler/developer-guide-reference/2023-0/list-of-runtime-error-messages.html
- flashes / 5 min unit issue