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.

Sign up for GitHub

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

Jump to bottom

TW polygon missing Lienchiang County and parts of Kinmen County (Wuqiu) #127

Closed
bdon opened this issue Jul 11, 2023 · 1 comment
Closed

TW polygon missing Lienchiang County and parts of Kinmen County (Wuqiu) #127

bdon opened this issue Jul 11, 2023 · 1 comment

Comments

@bdon
Copy link
Contributor

bdon commented Jul 11, 2023

Of the "TPKM" areas the TW polygon currently includes Taiwan, Penghu and the main island of Kinmen County, but there are a few islands missing. In keeping with the objectives of this library the motivation here is to power consistent locale information for mapping, so this doesn't include some unpopulated islands; the civilian population is 10,000+ of these in total and there is significant detail in OSM:

Wuqiu relation on OSM
Lienchiang (Matsu) relation on OSM

Screenshot of proposed polygon change with OSM Carto as a reference, using a minimal number of vertices:

Screenshot 2023-07-11 at 15 53 16
bdon added a commit to bdon/country-coder that referenced this issue Jul 11, 2023
@bdon
Adjust TW polygon to include Matsu and Wuqiu islands [rapideditor#127]
2e4924e
bhousel pushed a commit that referenced this issue Jul 11, 2023
@bdon @bhousel
Adjust TW polygon to include Matsu and Wuqiu islands [#127]
49ef989
@bhousel
Copy link
Collaborator

bhousel commented Jul 11, 2023

done in #127

@bhousel bhousel closed this as completed Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bhousel @bdon