-
Notifications
You must be signed in to change notification settings - Fork 169
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
area-based bioenergy policy #605
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks overall good to me. I just did some suggestions for reducing the footprint of the implementation.
Please don't forget to adjust the other realizations accordingly and to add a corresponding setting in default.cfg
@@ -5,6 +5,7 @@ | |||
*** | MAgPIE License Exception, version 1.0 (see LICENSE file). | |||
*** | Contact: [email protected] | |||
|
|||
p30_bioen_pol_ref_area(j)$(sum(ct, m_year(ct)) = s30_bioen_pol_ref_year) = pcm_land(j,"crop"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't that need to be in presolve? What happens if the start year should be 1995?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, postsolve is the right place for this. In case of 1995, cropland after optimization will be used as reference area for 2000, 2005 etc.
Co-authored-by: Jan Dietrich <[email protected]>
Co-authored-by: Jan Dietrich <[email protected]>
Co-authored-by: Jan Dietrich <[email protected]>
Co-authored-by: Jan Dietrich <[email protected]>
Co-authored-by: Jan Dietrich <[email protected]>
Co-authored-by: Jan Dietrich <[email protected]>
🐦 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 --> "compilation check"
Rscript start.R --> "test runs"
Rscript start.R --> "test runs"
📉 Performance changes 📈
🚨 Checklist for reviewer 🚨
CHANGELOG
is updated correctly