Update project mappings in piamInterfaces to correctly reflect AP emissions from MAgPIE that are now available in REMIND standalone and REMIND-MAgPIE coupled runs - #556
Conversation
… that are now available in REMIND standalone and coupled runs.
There was a problem hiding this comment.
Pull request overview
This PR updates emissions mappings across multiple IIASA template mapping files to reflect new/updated availability of REMIND-reported air pollutant emissions (incl. AFOLU components), and removes previously used temporary workarounds/warnings about missing MAgPIE-related air pollutant emissions.
Changes:
- Updated
mapping_ScenarioMIP.csvto map several air pollutant emissions variables (and AFOLU subcategories) toEmi|*REMIND variables, removing prior MAgPIE-side additions and dummy/“added manually” mappings for some categories. - Updated project mappings (
NAVIGATE,ARIADNE,AR6,climateassessment) to re-enable AFOLU air pollutant variables by mapping them to the correspondingEmi|*|AFOLUREMIND variables and removing outdated warning text. - Normalized index entries where previously blank in some templates (e.g., NAVIGATE, climateassessment) to make the mapping rows explicit.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| inst/mappings/mapping_ScenarioMIP.csv | Reworked air pollutant emissions mappings (incl. AFOLU breakdowns) to use REMIND `Emi |
| inst/mappings/mapping_NAVIGATE.csv | Restored AFOLU air pollutant emissions mappings (`Emi |
| inst/mappings/mapping_climateassessment.csv | Cleaned up air pollutant totals mappings (BC/CO/NH3/NOx/OC/SO2/VOC) by removing outdated warning notes and assigning explicit idx where applicable. |
| inst/mappings/mapping_ARIADNE.csv | Cleaned up air pollutant totals and restored AFOLU pollutant mappings where available, removing outdated warning notes. |
| inst/mappings/mapping_AR6.csv | Restored AFOLU air pollutant mappings (`Emi |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@merfort, @flohump, @dklein-pik: I am merging this now following Falk's approval. It would be great if you could nonetheless have a look so that you are up-to-date on the status. Thanks! If there are any issues, will be happy to correct in a follow-up PR. |
|
Thanks! I will have a look later today. |
Purpose of this PR
This PR is a follow-up of https://github.com/remindmodel/development_issues/issues/590 and addresses https://github.com/remindmodel/development_issues/issues/691.
Checklist:
renamed_piam_variables.csvto guarantee backwards compatibility.|+|notation consistently. This can be achieved automatically withupdatePlusUnit().It is recommended to have a look at the tutorial before submission.