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

More simplification to roads & boundary layers at zoom 9 #641

Closed
zerebubuth opened this issue Mar 29, 2016 · 3 comments
Closed

More simplification to roads & boundary layers at zoom 9 #641

zerebubuth opened this issue Mar 29, 2016 · 3 comments

Comments

@zerebubuth
Copy link
Member

Context, this comment:

I suspect the roads and boundary layers need additional geom simplifying.

We could also strip some properties off the boundary layer at low zooms (like name and the name variants) at zooms less than 8 as the zoom isn't large enough to provide screen space to do the labeling.

screen shot 2016-03-28 at 13 35 53

screen shot 2016-03-28 at 13 35 24

@nvkelso: what would be an appropriate level of simplification for the roads & boundaries features?

@nvkelso nvkelso added this to the v1.1.0 milestone Jun 6, 2016
@nvkelso nvkelso modified the milestones: v1.1.0, v1.2.0 Oct 5, 2016
@nvkelso nvkelso modified the milestones: v1.5.0, v1.6.0 Jun 27, 2017
@nvkelso
Copy link
Member

nvkelso commented Jun 27, 2017

This will be needed at both zoom 8 and 9 if we do the earlier NE to OSM switchover.

@nvkelso nvkelso modified the milestones: v1.5.0, v1.6.0 Nov 13, 2017
@nvkelso
Copy link
Member

nvkelso commented Nov 5, 2018

Let's try doubling the existing simplification and review.

Likley related to the generic simplify_and_clip function we pipe all layers thru.

@nvkelso
Copy link
Member

nvkelso commented Dec 14, 2018

We didn't see a huge drop in file size from this, but it did reveal an interesting bug we fixed.

No visual impact, closing.

@nvkelso nvkelso closed this as completed Dec 14, 2018
@ghost ghost removed the in review label Dec 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants