Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CCPP Tech Doc Updates for UFS SRW v2.2 release #67

Closed
8 of 9 tasks
ligiabernardet opened this issue May 10, 2023 · 2 comments
Closed
8 of 9 tasks

CCPP Tech Doc Updates for UFS SRW v2.2 release #67

ligiabernardet opened this issue May 10, 2023 · 2 comments

Comments

@ligiabernardet
Copy link
Collaborator

ligiabernardet commented May 10, 2023

Please edit/add as needed

  • Information on the various (and preferred) ways to add connect a scheme to CCPP, including pros/cons of using a wrapper versus pre- and post- interstitials
  • Best practices and guidelines wrt ways to allocate memory so schemes can be stateless (recommended but not required) (@dustinswales pls add as needed)
    • Suggest not using the term stateless because audience may not know what it is
    • Schemes should not use dynamic memory allocation on the HEAP
    • Schemes should not contain data that clashes when multiple non-interacting instances of the scheme are being used in one executable"
    • No allocations in module memory
  • More detailed code contribution instructions
  • Add note about problem fixed in Fix the broken error detection in GFS_phys_time_vary.fv3.F90 and pointer issues in c3 scheme ufs-community/ccpp-physics#106
  • "Single Column Model" --> "Single-Column Model"
@mkavulich
Copy link
Collaborator

Preliminary draft of chapter on adding a new scheme to CCPP here: https://docs.google.com/document/d/1DSRt_okwgc1MM6PRiy551RTV4rDXPqpUbKsy4dppf2Y/edit

@mkavulich mkavulich changed the title CCPP Tech Doc Updates for UFS SRW v3 release CCPP Tech Doc Updates for UFS SRW v2.2 release Oct 27, 2023
@mkavulich
Copy link
Collaborator

All items but one were complete; the incomplete one has been rolled into #70

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

No branches or pull requests

2 participants