change default logging parameters to be clearcut rather than reduced-impact

[ckoven]

Description:

These changes to the parameter file make harvest be clear-cut rather than reduced-impact logging. The argument is that that is more consistent with how the global land use datasets are treating logging. Ideally it would be better to have regionally- (and temporally-) differentiated logging parameters, but for initial global simulations, this endmember assumption is probably the place to start from.

Collaborators:

Expectation of Answer Changes:

This will change answers in all configurations that include logging.

Checklist

If this is your first time contributing, please read the CONTRIBUTING document.

All checklist items must be checked to enable merging this pull request:

Contributor

• The in-code documentation has been updated with descriptive comments
• The documentation has been assessed to determine if updates are necessary

Integrator

• FATES PASS/FAIL regression tests were run
• Evaluation of test results for answer changes was performed and results provided

Documentation

• Technical Note update:
• User's Guide update:

Test Results:

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

[glemieux]

FYI @sshu88 would you take a look at this when you have a moment, please?

[sshu88]

FYI @sshu88 would you take a look at this when you have a moment, please?

These revisions make sense for global default simulation. Thanks for working on it.

[mpaiao]

Hi @ckoven, I just checked the parameter updates, and they look good to me for making the clearcut the default.

In principle, logging characteristics could vary spatially, so these parameters could eventually be migrated to the land use files. This is very low priority though, I am not aware of any such dataset.