LANDIS-II-Foundation / Extension-Base-Harvest

Base Harvest extension for the LANDIS-II landscape change model.
https://sites.google.com/site/landismodel/extensions/base-harvest
Apache License 2.0
0 stars 7 forks source link

CompleteStandSpread requires 2 size parameters - not in User Guide #26

Closed brmiranda closed 7 years ago

brmiranda commented 7 years ago

When using CompleteStandSpread for the SiteSelection method, the extension expects 2 input parameters to follow. However the User Guide only lists one target size parameter. Based on the code it looks like the 2 expected parameters are minTargetSize and maxTargetSize. The User Guide basically implies that the minTargetSize would be 95% of the maxTargetSize, but at some point this became a separate input parameter. This probably just requires an update to the documentation. Note that this appears to apply equally to StandSpreading and PartialStandSpreading.

rmscheller commented 7 years ago

Good catch - thanks!

rmscheller commented 7 years ago

Corrected and uploaded. Will be included in next release.