Move run_has_transient_landcover function into a fire-specific module #1560
Labels
bfb
bit-for-bit
code health
improving internal code structure to make easier to maintain (sustainability)
good first issue
simple; good for first-time contributors
The function run_has_transient_landcover is defined here:
CTSM/src/dyn_subgrid/dynSubgridControlMod.F90
Line 350 in 2a88e90
However, as noted here:
#1546 (comment)
there are subtleties about which aspects of transient landcover should and should not be included in this function. As I noted in the above comment:
Probably what we should do is to move this function into the fire code, and make it fire-specific, since otherwise it isn't obvious why this should include some transitions but not others.
As noted in #52 there are some broader issues with the mix of human-induced and natural landcover transitions, but those bigger issues would probably be a pain to fix, so for now I think it is sufficient to move this run_has_transient_landcover function into the fire code with comments saying that this just includes potentially human-induced aspects of landcover change.
The text was updated successfully, but these errors were encountered: