Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of this PR
This pull request has two main purposes. The first one is to add the Sticky realization to the Factor costs module. Here, the capital relocation flexibility is reduced to improve crop production dynamics at different spatial scales. The second purpose is to modify how costs are calculated in the modules where investment decisions are taken.
Performance loss/gain from the current default behavior
Type of change
How Has This Been Tested?
test_runs.R
).test_runs.R
but with the changes from PR.Additions or Changes to default configuration (default.cfg):
The new Sticky realization was listed in the module_38 section of the default.cgf.
Changes are deletion or updates to the existing model components in default config
Checklist:
CHANGELOG.md
goxygen::goxygen()
for the new/updated parts.magpie4
R library for post processing of model output (ideally backward compatible).Special comments/warnings
We are still learning about the "sticky" effects and limitations. Please be cautious when using it for different region settings.