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.
🐦 Description of this PR 🐦
🔧 Checklist for PR creator 🔧
Label pull request from the label list.
Self-review own code
magpie4
R library has been updated accordingly and backwards compatible where necessary.scenario_config.csv
has been updated accordingly (important ifdefault.cfg
has been updated)Document changes
CHANGELOG.md
goxygen::goxygen()
and verify the modified code is properly documentedPerform test runs
Rscript start.R --> "test runs"
📉 Performance changes 📈
Slightly better runtime
Some results:
![EFP_tau](https://user-images.githubusercontent.com/39262100/207312318-ec631682-425f-4744-9f46-389b56e51473.PNG)
Tau unchanged
Area actually irrigated
![EFP_AAI](https://user-images.githubusercontent.com/39262100/207312387-b34db864-de53-4cf1-93b6-a8bae6609c3e.PNG)
Withdrawal comparing EFP off with EFP on (old and new)
![EFPwithdrawal](https://user-images.githubusercontent.com/39262100/207312444-51781b25-9e80-4840-9046-bd1bd8e012bb.PNG)
Withdrawal with example of different start and end year compared to previous default
![withdrawalNEWSCEN](https://user-images.githubusercontent.com/39262100/207312518-32cd841c-e63f-423b-afc3-b2507a58ec4f.PNG)
🚨 Checklist for reviewer 🚨
CHANGELOG
is updated correctly