Skip to content
This repository was archived by the owner on Apr 23, 2024. It is now read-only.

fix(map): fix water or multipolygons in general#41

Open
pwambach wants to merge 3 commits intomasterfrom
fix/water
Open

fix(map): fix water or multipolygons in general#41
pwambach wants to merge 3 commits intomasterfrom
fix/water

Conversation

@pwambach
Copy link
Contributor

@pwambach pwambach commented Jun 8, 2018

No description provided.

@pwambach pwambach requested a review from aemkei June 8, 2018 13:54
Copy link
Member

@aemkei aemkei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find!

Only make sure to exclude the ferries and I'm fine with that.


const API_KEY = 'fApLQBTwQbaIclmV0CoOQA';
const TILE_BASE_URL = 'https://tile.nextzen.org/tilezen/vector/v1/256/all/';
const EXCLUDE_ROAD_TYPES = ['ferry', 'path', 'minor_road', 'rail'];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least we should exclude the 'ferry' type here.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pwambach Can you please have a look (next Friday)?

@pwambach
Copy link
Contributor Author

Excluded ferries and pedestrians. Looks better now.
Please note that the mapbox key is still hard coded into the tile url. Should be removed before making this repo public.

screen shot 2018-07-13 at 10 07 50

@pwambach
Copy link
Contributor Author

Ah wait, there is still an error when merging the multiPolgons (the horizontal lines over the elbe)

@pwambach
Copy link
Contributor Author

screen shot 2018-07-13 at 11 54 14

The problem are the tile borders in the MultiPolygons. Not sure how to fix that...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants