-
Notifications
You must be signed in to change notification settings - Fork 45
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
Infinite CCS potential for DEU reported #561
Comments
@orichters pointed towards these lines (thank you!): @amerfort and @fschreyer: Is this line still required given the update of the near-term CCS bounds? I believe the DEU limit is no longer envisioned to be 0 in 2030? If this should still be an option for different scenarios, we might want to change it to a switch? |
If Germany is covered by the new CCS bounds, which I think is the case, this can be removed. |
@tabeado Can we fix that before the release, probably just removing those two lines? |
Summary of a brief discussion with Felix:
Felix takes it to the REMIND-EU meeting to make a decision. |
Was any decision made?
|
I don't see how it relates to this line. The run you show does not have it anymore. Shouldn't summing over all elements of an upper bounds always give |
In
In
This added values to many more |
Hi, thanks for picking this up again, I somehow had thought it was solved. |
Thanks for figuring out the bug, Felix and Oli! It only occurred for NPis. I changed the reportEmi in this PR: #654 When accepted, I think we can close this issue. |
This
remind2/R/reportEmi.R
Lines 1634 to 1635 in ee372c4
does not work for
DEU
(and by extensionEUR
,EU27
, andWorld
). For an example see/p/projects/remind/modeltests/remind/output/SSP2EU-EU21-NPi-AMT_2024-03-01_22.12.41/REMIND_generic_SSP2EU-EU21-NPi-AMT_withoutPlus.mif
For some reason,
vm_co2ccs.up(DEU,pcc) and vm_co2ccs.up(DEU,pco)
areInf
, which of course sums up toInf
. Either figure out why that is on the REMIND/GAMS side, or hereThe text was updated successfully, but these errors were encountered: