mledoze / countries

World countries in JSON, CSV, XML and Yaml. Any help is welcome!
https://mledoze.github.io/countries/
Open Data Commons Open Database License v1.0
5.98k stars 1.27k forks source link

Adding GB Subdivisions Per ISO 3166-2:GB #357

Open rubengmurray opened 5 years ago

rubengmurray commented 5 years ago

I have a use case for England, Scotland, Wales & Northern Ireland to be identified individually rather than blanket 'GB' attribution.

I note that this was mentioned quite a while ago in https://github.com/mledoze/countries/issues/6#issuecomment-27485840 but can't see there's been any discussion since...

@mledoze I'd be happy to contribute. Sounds like there were some hierarchy changes suggested for implementation, were they started? Did you have any thoughts on this specific issue? Could we put them at the top-level and retain shared information across the 4?

mledoze commented 4 years ago

Hello @rubengmurray thank you for bringing up this issue again. It is time that we provide a clean solution to it.

I'd be happy to contribute. Sounds like there were some hierarchy changes suggested for implementation, were they started?

Yes there were, but no work was made on this subject. Your contribution is more than welcome! As you can see—with my delay to answer you—I lack the time to implement all the ideas I have for this project, especially this one.

There are two subjects here:

  1. add GB subdivisions per ISO 3166-2:GB, which would basically mean to implement ISO 3166-2 for all countries because it would not make sense to do it for only one country

  2. refactor the implementation of countries.json into separate files

If you are willing to contribute, I would prefer to start with 2. since it is something that I have wanted for a long time. Once it is done, I think we will be in a better position to start thinking about 1..

What do you think?

jjlharrison commented 4 years ago

I may be able to help with this.

I have compiled a set of files containing subdivision information for each country, but I stalled when trying to come up with a way of structuring the hierarchy of data. I think it might be better to just have a flat list for each country.

I agree that adding this information would greatly increase the size of the file and should probably be split into separate files.

I need the information for a project due in January/February of the new year, so I should be able to help in the near future.

rubengmurray commented 4 years ago

Hi @mledoze

@jjlharrison what did you end up doing for your project?

Splitting them out into separate files is probably a good idea. I've knocked something up on a separate branch splitting the json out and I'm happy to push to this repo to share and work on if you're still interested? I don't have a ton of time either but can share the load on a branch or two.

Cheers

rubengmurray commented 4 years ago

Initial split: https://github.com/rubengmurray/countries/tree/individual-files/country-data

mledoze commented 4 years ago

Hello @rubengmurray thank you for your work, yes I am still interested!

I have looked at your branch on your fork. I would like to change some things:

What do you think? You said that you don't have a lot of time, I could continue your work if you want.

I will also need to update the build code to accommodate these changes.

rubengmurray commented 4 years ago

Sounds good to me - I think this is it:

https://github.com/mledoze/countries/pull/393

Maybe we can merge this PR into a separate branch on this repo so you can do the build work?

Cheers

rubengmurray commented 3 years ago

@mledoze any appetite for https://github.com/mledoze/countries/pull/393 ?

Seems to have a few merge conflicts now... may be best to close if not part of the plans for the repo?