ESCOMP / ESCOMP/atmospheric_physics

nucleati future cleanup: remove call_frm_zm_in

Open
#420 0 comments 0 reactions 0 assignees View on GitHub
aerosols cleanup
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.