remindmodel / remind

REMIND - REgional Model of INvestments and Development
Other
99 stars 129 forks source link

make cement process emissions variable for conopt #1673

Closed JakobBD closed 4 months ago

JakobBD commented 4 months ago

Purpose of this PR

So far, cement process emissions were fixed to a value based on the previous iteration's ue_cement. ue_cement therefore only saw energy-related emissions as an incentive for reduction.

This PR makes cement process emissions a function of ue_cement in the current iteration, such that CONOPT sees the incentive to reduce ue_cement.

I still have to check and clean up the fixing of vm_macBase and vm_emiIndBase in core/presolve.gms, as process cement CCS cost enters the budget equation twice right now.

TBD if we like the results, and if we want to do some ad-hoc adaptation, e.g. to CES elasticities.

Type of change

(Make sure to delete from the Type-of-change list the items not relevant to your PR)

Checklist:

Further information (optional):

image

0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q commented 4 months ago

I tested this too, same result (unsurprisingly). image Weird to see these differences between AMT and stock develop runs, though. I do not see any point in adjusting CES elasticities over this.

JakobBD commented 4 months ago

Doing final tests on this after separating the fixed_shares refactoring (same directory as above, newest two runs). From my side ready to merge once these tests have passed.

JakobBD commented 4 months ago

Tests came back fine, ready to merge from my point of view.