Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added PRISMA mapping (based on ScenarioMIP) #436

Merged
merged 4 commits into from
Jan 24, 2025

Conversation

dbantje
Copy link
Contributor

@dbantje dbantje commented Jan 22, 2025

Purpose of this PR

This adds a mapping for the PRISMA project. The mapping is based on the ScenarioMIP mapping. For now, additional variables for material demands, material stocks, and scraps are not mapped.

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.

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

@dbantje dbantje marked this pull request as draft January 24, 2025 11:54
@dbantje dbantje marked this pull request as ready for review January 24, 2025 11:55
@dbantje
Copy link
Contributor Author

dbantje commented Jan 24, 2025

@orichters , @fbenke-pik Can't assign reviewers here. Could you review this PR?

@dbantje dbantje requested a review from orichters January 24, 2025 12:34
@dbantje dbantje merged commit 6177003 into pik-piam:master Jan 24, 2025
1 check passed
@dbantje dbantje deleted the prisma_submission branch January 24, 2025 12:48
@orichters
Copy link
Contributor

For the record: We added the Test line because mappings with no single piam_variable fail the tests in various ways.

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.

2 participants