-
Notifications
You must be signed in to change notification settings - Fork 133
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
fix recycling of implicit tax from BECCS qtty target in regipol #1745
base: develop
Are you sure you want to change the base?
Conversation
ea07ee4
to
f4c60d1
Compare
Ok, that does not work at all.
does not contain variables, but just things that are updated in postsolve only. So you basically add and substract the exact same value, thus the tax has no effect at all in this case. Would probably mean turning And somehow I just implemented the recycling, not the tax itself. Why the hell did that work at all? |
)$(sameas(qttyTarget,"CCS") AND sameas(qttyTargetGroup,"all")) | ||
+ | ||
(( !! Supply side BECCS | ||
sum(emiBECCS2te(enty,enty2,te,enty3),vm_emiTeDetail.l(t,regi,enty,enty2,te,enty3)) |
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.
you need to remove the .l here so the model knows there is a trade-off between the tax and the level of vm_emiTeDetail here.
(( !! Supply side BECCS | ||
sum(emiBECCS2te(enty,enty2,te,enty3),vm_emiTeDetail.l(t,regi,enty,enty2,te,enty3)) | ||
!! Industry BECCS (using biofuels in Industry with CCS) | ||
+ sum((emiMkt,entySe,secInd37,entyFe)$entySeBio(entySe), pm_IndstCO2Captured(t,regi,entySe,entyFe,secInd37,emiMkt)) |
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.
In order to consider the industry BECCS tradeoff, you should replace the parameters calculated in the postsolve by the variables used to calculate them.
However this will be most probably a quite non-linear equation, and I am not sure how big is the importance of the industry BECCS in the target you want to achieve.
a255294
to
aaf2ffc
Compare
Purpose of this PR
Type of change
Checklist:
remind2
where it was neededforbiddenColumnNames
in readCheckScenarioConfig.R in case the PR leads to deprecated switchesFAIL 0
in the output ofmake test
):[ FAIL 0 | WARN 0 | SKIP 6 | PASS 83 ]
CHANGELOG.md
has been updated correctly