Closed 0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q closed 1 week ago
@orichters pointed towards these lines (thank you!): https://github.com/remindmodel/remind/blob/854dfea96f49906db764b6b1f3dcf7e3fdf59c06/modules/47_regipol/none/bounds.gms#L24 https://github.com/remindmodel/remind/blob/854dfea96f49906db764b6b1f3dcf7e3fdf59c06/modules/47_regipol/regiCarbonPrice/bounds.gms#L54
@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.
Felix takes it to the REMIND-EU meeting to make a decision.
Was any decision made?
SSP2-EU21-NPi-AMT_2024-08-16_22.12.58/REMIND_generic_SSP2-EU21-NPi-AMT_withoutPlus.mif
Model Scenario Region Variable Unit 2005 2010 2015 2020 2025 2030
REMIND SSP2-EU21-NPi-AMT DEU Carbon Management|Storage|Maximum annual CO2 storage potential Mt CO2/yr Inf Inf Inf Inf Inf Inf
REMIND SSP2-EU21-NPi-AMT EU27 Carbon Management|Storage|Maximum annual CO2 storage potential Mt CO2/yr Inf Inf Inf Inf Inf Inf
REMIND SSP2-EU21-NPi-AMT EUR Carbon Management|Storage|Maximum annual CO2 storage potential Mt CO2/yr Inf Inf Inf Inf Inf Inf
REMIND SSP2-EU21-NPi-AMT World Carbon Management|Storage|Maximum annual CO2 storage potential Mt CO2/yr Inf Inf Inf Inf Inf Inf
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 INF
because this is what an upper bound is by default? Without looking deeper, I don't quite get why it only occurs for Germany. I would say that we should just assign elements of cco2.ico2.ccsinje.1
in this line as this is what is used as CCS potential and that should do it.
In core/bounds.gms
, we have
vm_co2CCS.up(t,regi,"cco2","ico2","ccsinje","1")
In modules/47_regipol/none/bounds.gms
and earlier in regipol, the assignment was
vm_co2CCS.up(t,regi,"cco2","ico2",te,rlf)
This added values to many more te
and rlf
instead of just ccsinje
and 1
, and then this sum resulted in Inf. I agree, just use ccsinje.1
in reportEmi. I would still remove the line in modules/47_regipol/none/bounds.gms
as well.
Hi, thanks for picking this up again, I somehow had thought it was solved. Unfortunately I can make the adjustment only after I return from my vacation in 2 weeks. If you want it fixed before that, please go ahead.
Thanks for figuring out the bug, Felix and Oli! It only occurred for NPis.
I changed the reportEmi in this PR: https://github.com/pik-piam/remind2/pull/654
When accepted, I think we can close this issue.
This https://github.com/pik-piam/remind2/blob/ee372c4431f0b6f9e7d6789b1fae05f64d288a19/R/reportEmi.R#L1634-L1635 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 here