Skip to content

add Policy Cost to ScenarioMIP mapping, taken from AR6 mapping - #569

Merged
lecfab merged 3 commits into
pik-piam:masterfrom
lecfab:master
Oct 21, 2026
Merged

lecfab merged 3 commits into
pik-piam:masterfrom
lecfab:master

Conversation

@lecfab

@lecfab lecfab commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Purpose of this PR

Policy Cost variables are missing in scenarioMIP mapping (used for NGFS6) while they are in AR6 mapping (used for NGFS5).
This PR adds them.

Checklist:

  • I did not use Excel to open csv files or checked that no side-effects occur (changed values, many new quotation marks, …)
  • I added any renamed piam_variable to renamed_piam_variables.csv to guarantee backwards compatibility.
  • For REMIND variables, I used |+| notation consistently. This can be achieved automatically with updatePlusUnit().

It is recommended to have a look at the tutorial before submission.

@RahelMA RahelMA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lecfab

lecfab commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

could you please also add this https://github.com/lecfab/piamInterfaces/blob/edf80c4240270f3c06e72f3a601b0f72645284bb/inst/mappings/mapping_ECEMF.csv#L690
Policy Cost|Area under MAC Curve;billion EUR_2020/yr;;;;;;;

As far as I see, no REMIND variable is available to feed into Policy Cost|Area under MAC Curve. Should someone add it? is there a more specific definition of this variable?

In the meantime, we could merge this PR without.

@lecfab
lecfab requested a review from RahelMA October 21, 2026 12:02
@lecfab
lecfab dismissed RahelMA’s stale review October 21, 2026 12:02

will be addressed later

@lecfab
lecfab merged commit 96ebfcf into pik-piam:master Oct 21, 2026
1 check passed
@lecfab

lecfab commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

@lea-hayez @RahelMA you may add your reporting requests for PRISMA in the new issue:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants