Require ccpp_physics_initialize to list all suites?
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 25
- Forks
- 67
- Avg merge
- 3d 8h
- Merged PRs (30d)
- 2
Description
Currently <host>_ccpp_physics_initialize takes a single suite name as its input.
Also, currently, the interface for registering constituents is a separate call which takes a list of suites.
Proposal: require the host model to pass all active suite names to <host>_ccpp_physics_initialize.
The advantage is that <host>_ccpp_physics_initialize could then call ccpp_register_constituents saving a new host model calling requirement.
This would be implemented by adding an interface to <host>_ccpp_physics_initialize to take an array of suite names. For host models that only initialize a single suite, the current interface will remain.
Thoughts? @climbfuji?
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the current _ccpp_physics_initialize interface and the separate ccpp_register_constituents call. Confirm how suite names are passed today, then define the compatibility behavior for single-suite and multi-suite initialization; done means the interface and host-model calling requirements are consistently specified.
Written by the indexing model from the issue text.
Assessment
- Domain
- backend-api-design
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100