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

Update al-tirana.geojson #149

Merged
merged 1 commit into from
May 15, 2018
Merged

Update al-tirana.geojson #149

merged 1 commit into from
May 15, 2018

Conversation

jbelien
Copy link
Contributor

@jbelien jbelien commented May 15, 2018

Fix Tirana's feature geometry.
I simplified the geometry from https://www.openstreetmap.org/relation/1250106 ; if needed I can simplify it more.

Fix Tirana's feature geometry.
I simplified the geometry from https://www.openstreetmap.org/relation/1250106 ; if needed I can simplify it more.
@bhousel
Copy link
Member

bhousel commented May 15, 2018

I simplified the geometry from https://www.openstreetmap.org/relation/1250106 ; if needed I can simplify it more.

The polygon doesn't really need to be very detailed, but the point of this is to show people editing inside the polygon that there is a meetup in Tirana, so that's why I usually try to make the bounding volume more like "within reasonable driving distance" rather than "city limits". Do you still want me to merge it?

@jbelien
Copy link
Contributor Author

jbelien commented May 15, 2018

Okay, I get it :)

I still think it's worth merging this one.

@bhousel bhousel merged commit d165175 into osmlab:master May 15, 2018
@bhousel
Copy link
Member

bhousel commented May 15, 2018

Okay, I get it :) I still think it's worth merging this one.

Thanks, just merged 👍

@jbelien jbelien deleted the patch-1 branch May 15, 2018 22:32
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