iiasa / message-ix-models

Tools for the MESSAGEix-GLOBIOM family of models
https://docs.messageix.org/models
Apache License 2.0
17 stars 33 forks source link

Add node code lists R17 and R20 #108

Closed SiddharthJoshi-Git closed 1 year ago

SiddharthJoshi-Git commented 1 year ago

This PR adds two new regional mappings for GUIDE project and YSSP'23 work related to it. The R17 regional mapping is for YSSP work and R20 for overall GUIDE project.

How to review

PR checklist

khaeru commented 1 year ago

@SiddharthJoshi-Git, thanks for starting this PR.

As next steps, you should:

SiddharthJoshi-Git commented 1 year ago

@SiddharthJoshi-Git, thanks for starting this PR.

  • I updated the PR description, in particular to re-add the "PR checklist," which is not optional. Please re-read the comments inline to understand how to use ~strike through~ if you think a particular item should not be needed for a certain PR (though in this case, I think all are relevant).

As next steps, you should:

  • Bring your branch up to date with iiasa:main. You can use the provided button above: "Update branch > update with rebase" (and then git pull --rebase on your local system to mirror the changes).
  • Look at the checks/tests that are failing. (Click the "Details" next to the failing jobs.) Try to make a guess as to why, and if possible adjust or expand the tests so that they pass.

HI @khaeru,

The 'ALL' tests are failing due to licensing error from GAMS. As per some suggestions I received, can this be due to me starting a PR from my fork. Should I restart a PR by making a branch within the message_ix_models (IIASA) to remedy the licensing errors?

Best, Sid

khaeru commented 1 year ago

Should I restart a PR by making a branch within the message_ix_models (IIASA) to remedy the licensing errors?

Yes, that sounds like a good idea. A nice branch name to use would be something like enh/node-R17-R20 (for "enh"ancement).

You can copy the raw Markdown from this PR's description into the new one; saves re-writing.