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

Drop all_networks and all_shield_texts from low and mid zooms #1642

Closed
nvkelso opened this issue Sep 25, 2018 · 1 comment
Closed

Drop all_networks and all_shield_texts from low and mid zooms #1642

nvkelso opened this issue Sep 25, 2018 · 1 comment

Comments

@nvkelso
Copy link
Member

nvkelso commented Sep 25, 2018

To decrease file size let's drop all_networks and all_shield_texts from low and mid-zooms when there's often not room to display this information.

  • highway: less than zoom 10
  • major_road: less than zoom 12
  • other kinds: less than zoom 14

NOTE: MVT format currently doesn't include the array objects while GeoJSON and TopoJSON do.

@nvkelso
Copy link
Member Author

nvkelso commented Dec 14, 2018

No obvious regressions (in MVT) here, closing.

(Would later yield savings if we write out nested props as strings into MVT like some software does.)

@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