-
Notifications
You must be signed in to change notification settings - Fork 177
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
Elastic Food Demand and Tax recycling #767
Conversation
removed all realization of food demand module, as it is outdated.
…develop # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
The bugfixing for elastic demand looks good to me and Jiaqi. |
Thanks for looking into it! |
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.
Thanks for these improvement.
Looks fine overall.
Please mention in the CHANGELOG that you removed modules/15_food/anthropometrics_jan18
You changed 3 renv
files. Please double check
modules/15_food/anthropometrics_jan18 deleted
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.
Thanks for clarifying.
Before you proceed you need to sort out merging conflicts.
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 good to me, just two small questions
…develop # Conflicts: # CHANGELOG.md # modules/15_food/anthropometrics_jan18/declarations.gms # modules/15_food/anthropometrics_jan18/equations.gms # modules/15_food/anthropometrics_jan18/input.gms # modules/15_food/anthropometrics_jan18/intersolve.gms # modules/15_food/anthropometrics_jan18/postsolve.gms # modules/15_food/anthropometrics_jan18/preloop.gms # modules/15_food/anthropometrics_jan18/presolve.gms # modules/15_food/anthropometrics_jan18/realization.gms # modules/15_food/anthropometrics_jan18/scaling.gms # modules/15_food/anthropometrics_jan18/sets.gms
Co-authored-by: Jan Dietrich <[email protected]>
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 good, thank you.
@@ -178,6 +175,11 @@ parameters | |||
* country-specific scenario switch | |||
p15_country_switch(iso) Switch indicating whether country is affected by diet scenarios (1) | |||
|
|||
* elastic demand | |||
p15_tax_recycling(t,iso) Tax revenue recycling to households (USD17PPP per capita) |
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.
The name of the switch 's15_tax_recycling' that determines how GHG tax revenues are recycled is very similar to the name of the parameter 'p15_tax_recycling(t,iso)'.
Would it be an idea to rename one of them to make the code easier to read?
all requested changes done. now want to merge
🐦 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
Low risk:
Rscript start.R --> "compilation check"
Medium risk:
Rscript start.R --> "test runs"
High risk:
Rscript start.R --> "test runs"
test run scenario setup:
3 taxation scenarios with full tax recycling (1), medium (0.5) and none (0)
A scenario without taxation (Ref-1)
A scenario without taxation and without elastic demand (Ref-inelastic)
total calories: no major changes
some changes in ruminant meat consumption in SSA
Taxation with tax recycling has almost no effect, only if taxes are not recycled consumption declines
food expenditure rises in taxation scenarios.
Price drop in 2025, as the model switches from exogenous prices to endogenous prices in the elastic version of the model. Drop would be lower if our prices would be more realistic.
📉 Performance changes 📈
🚨 Checklist for reviewer 🚨
CHANGELOG
is updated correctly