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 Pontevedra Spain; Added local Spelling for La Coruna, Spain #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ignaciogc
Copy link

No description provided.

@ocram
Copy link
Contributor

ocram commented Oct 23, 2018

Thank you very much!

As for cities, our source considers only “capitals, major cities and towns, plus a sampling of smaller towns in sparsely inhabited regions”. They “favor regional significance over population census in determining [their] selection of places”. So Pontevedra may be too small for them under the criteria described.

But I don’t see why we shouldn’t include more cities than our source data set does, in particular Pontevedra. So it should be fine to include it.

By the way, what’s your source for the coordinates? Wikipedia links to the slightly different -8.633333 42.433333.

Other names for La Coruna are included in the updated data of our source. So we should definitely compile an updated data set from the source first.

Performing that update should be done first, anyway, to avoid overwriting any other changes.

@ignaciogc
Copy link
Author

Hi @ocram , I used https://www.gps-coordinates.net/ to get the GPS coordinates of Pontevedra, Spain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants