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

Adding nigerian zones and states #5

Merged
merged 4 commits into from
Apr 5, 2024
Merged

Conversation

erikfilias
Copy link
Contributor

@erikfilias erikfilias commented Mar 15, 2024

Introduction

This pull request aims to incorporate the six geopolitical zones and thirty-six states of Nigeria, along with the Federal Capital Territory, into the project's nomenclature. This addition will facilitate more region-specific analyses, data segmentation, and user interface enhancements tailored to Nigerian users or studies focusing on Nigeria.

Changes Made

  • A new list of all Nigerian states and their respective zones was added to the "african_subregions" yaml file.

I look forward to your feedback @phackstock and @sandrinecharousset.

Copy link
Contributor

@phackstock phackstock left a comment

Choose a reason for hiding this comment

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

Looks good to me in principle thanks @erikfilias.
Just one small issue as explained below as well. The region names need to be unique for the Scenario Explorer and Niger already exists as a country. Therefore I'd recommend the prefix mentioned below.

definitions/region/african_subregions.yaml Outdated Show resolved Hide resolved
definitions/region/african_subregions.yaml Outdated Show resolved Hide resolved
@erikfilias erikfilias requested a review from phackstock April 3, 2024 13:31
@phackstock phackstock merged commit 371d680 into iiasa:main Apr 5, 2024
2 checks passed
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