python-iso3166 icon indicating copy to clipboard operation
python-iso3166 copied to clipboard

Outdated list of countries

Open detrin opened this issue 1 year ago • 5 comments

For example, Kosovo is no longer in https://en.wikipedia.org/wiki/ISO_3166-1

I could open a PR. I think even better is create a job that will check the list with scraping wiki on weekly basis. I will try to prepare that

detrin avatar Jun 06 '24 09:06 detrin

I prepared a repo where you can use the CLI to fetch the newest ISO 3166 list or use the countries.json file. https://github.com/detrin/download-iso3166-list

detrin avatar Jun 07 '24 13:06 detrin

Opened PR https://github.com/deactivated/python-iso3166/pull/38

detrin avatar Jun 07 '24 21:06 detrin

also opened PR for the script check https://github.com/deactivated/python-iso3166/pull/39

detrin avatar Jun 07 '24 21:06 detrin

Hi @detrin,

Thanks for the interest. The XK alpha-2 code is a defacto standard in use by a number of international organizations (see Wikipedia for a partial list).

I don't intend to remove the entry, however, I would be open to a PR that adds a status field to each entry.

deactivated avatar Jun 12 '24 09:06 deactivated

@deactivated I see you closed both my PRs, so my effort goes for nothing 😐

As of now the repo itself depends on people noticing you about the ISO updates in issues. I have noticed that it is normal that repos containing ISO 3166 have inconsistencies in it, that have to be resolved manually. Especially this package is for python, which is commonly used in feature engineering in datascience.

Ad Kosovo, Kosovo is not in ISO 3166-1 list, hence if you present this repo as ISO 3166 list and you have Kosovo in it, it is not really true information. The result is that someone my consider Kosovo in the list, someone may not, but if the python package is named iso3166 I believe it should contain true information about ISO 3166. You could create optional argument that would be opt-in and not opt-out for ISO 3166 - extended for example and explicitly mention that with Kosovo case in README.

I think there are those two issues 1) updating/checking and 2) consistency.

detrin avatar Jun 12 '24 09:06 detrin