ESCOMP / ESCOMP/atmospheric_physics
nucleati future cleanup: remove call_frm_zm_in
- Dominant language
- Fortran
- Stars
- 12
- Forks
- 38
- Avg merge
- 11h 16m
- Merged PRs (30d)
- 5
Description
During the review of https://github.com/ESCOMP/atmospheric_physics/pull/411, @nusbaume originally wrote:
> You should certainly double-check me, but is call_frm_zm_in ever not .false. in CAM? If so then I might vote to remove it (or add a note in the "future work" issue to remove it later).
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing pull request 411 and searching the repository for call_frm_zm_in and its callers. Check whether it can ever be true in CAM or related configurations; done means either removing the unused path safely or documenting why the cleanup must remain future work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fortran
- Domain
- hpc
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100