Fix bug with switch cm_inco0Factor for scaling investment cost of technologies, investment cost in policy runs are not overwritten by reference gdx in 05_initialCap anymore #1390
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of this PR
This fixes a bug for changing the investment cost in core/datainput with switches like
cm_inco0Factor
. Those changes were overwritten in policy runs at the end of the 05_initialCap module since the earlier change that initialCap is not run in policy runs anymore. Now, pm_inco0_t is not loaded from the input_ref.gdx anymore in policy runs. It is only loaded in case of cm_techcost = GLO where initialCap has an actual influence on pm_inco0_t. In this case, only the technologies modified by initialCap code are overwritten (ngcc etc.). I added a note to the above switches that those technologies are in this case still not affected by the switches. In all other cases, the switch should work as expected.Type of change
(Make sure to delete from the Type-of-change list the items not relevant to your PR)
Checklist:
FAIL 0
in the output ofmake test
)CHANGELOG.md
has been updated correctlyFurther information (optional):
Test runs are here:
/p/tmp/schreyer/Modeling/remind/Current/output/test_inco0_switch_glo_2023-08-29_11.42.31
Comparison of results (what changes by this PR?):