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

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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.

@@ -6,8 +6,6 @@ import Protobuf from 'pbf';

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