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

Add local zone #6287

Merged
merged 7 commits into from
Mar 8, 2022
Merged

Add local zone #6287

merged 7 commits into from
Mar 8, 2022

Conversation

Mannivu
Copy link
Contributor

@Mannivu Mannivu commented Mar 7, 2022

I've added local .geojson to better locate potential BBC Roma banks.

Mannivu and others added 4 commits February 17, 2022 12:58
Removed matchname from BCC Roma: in Italy there are quite a lot of BCC and only a few are part of BBC Roma brand. Since I don't think there's the ability to limit for just a region but the NSI only works on national scale I think it's better to delete the matchname
@arch0345
Copy link
Collaborator

arch0345 commented Mar 7, 2022

Thanks for adding these geojsons, however, I recommend drawing these by hand instead with geojson.io using the polygon tool as suggested in this wiki article since we try to use as few points as possible, see #4784 (comment). There's no strict limit, but I would try to use at less than 100 points per geojson.

Uploaded simplified version of the polygons
Uploading simplified versions of the polygons
@Mannivu
Copy link
Contributor Author

Mannivu commented Mar 8, 2022

@arch0345 I've updated simplified version of the geojsons. I hope they are fine.

@arch0345
Copy link
Collaborator

arch0345 commented Mar 8, 2022

Thanks @Mannivu!

@arch0345 arch0345 merged commit 28c3241 into osmlab:main Mar 8, 2022
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