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

Upgrades for web map display (add min_zoom) #202

Closed
nvkelso opened this issue Jul 7, 2017 · 6 comments
Closed

Upgrades for web map display (add min_zoom) #202

nvkelso opened this issue Jul 7, 2017 · 6 comments
Milestone

Comments

@nvkelso
Copy link
Owner

nvkelso commented Jul 7, 2017

Commonly used now:

As used by Tilezen, Mapzen's vector tile stack.

  • ne_10m_admin_0_boundary_lines_land
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_10m_admin_0_boundary_lines_map_units
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_10m_admin_1_states_provinces_lines
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_10m_coastline
    • min_zoom
  • ne_10m_lakes
    • min_zoom
  • ne_10m_land
    • min_zoom
  • ne_10m_ocean
    • min_zoom
  • ne_10m_playas
    • min_zoom
    • character encoding funk
  • ne_10m_populated_places
    • min_zoom
    • character encoding funk
  • ne_10m_roads
    • min_zoom
  • ne_10m_urban_areas
    • min_zoom
  • ne_50m_admin_0_boundary_lines_land
    • min_zoom
    • min_label
    • min_zoom for the 10m polygons
  • ne_50m_admin_1_states_provinces_lines
    • min_zoom
    • min_label
    • min_zoom for the 10m polygons
  • ne_50m_coastline
    • min_zoom
  • ne_50m_lakes
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_50m_land
    • min_zoom
  • ne_50m_ocean
    • min_zoom
  • ne_50m_playas
    • min_zoom
  • ne_50m_urban_areas
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_110m_admin_0_boundary_lines_land
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_110m_coastline
    • min_zoom
  • ne_110m_lakes
    • min_zoom
    • min_zoom for the 10m polygons
  • ne_110m_land
    • min_zoom
    • oops: bad scalerank (to 0 from 1), bad featureclass (to Land from Country)
  • ne_110m_ocean
    • min_zoom

Should be used more often:

  • ne_10m_geography_regions_points
    • min_zoom basic
    • character encoding funk
  • ne_10m_geography_regions_elevation_points
    • min_zoom basic 0-7
    • min_zoom zooms 8++
    • character encoding funk (around 42 features, rescued from v1.4)
  • ne_10m_lakes_north_america
  • ne_10m_lakes_europe
    • min_zoom
    • character encoding funk (present since v1.2 release, problem with EU source?)
  • ne_10m_rivers_europe
    • min_zoom
    • character encoding funk
  • ne_10m_rivers_north_america
    • min_zoom
  • ne_10m_minor_islands
    • min_zoom
  • ne_10m_reefs
    • min_zoom
  • ne_10m_rivers_lake_centerlines
    • min_zoom
  • ne_10m_rivers_lake_centerlines_scale_rank
    • min_zoom
  • ne_10m_geography_regions_polys
    • min_zoom
  • ne_10m_geography_marine_polys
    • min_zoom
  • ne_10m_admin_0_disputed_areas
    • min_zoom
  • ne_10m_antarctic_ice_shelves_polys
    • min_zoom
    • character encoding funk
  • ne_10m_glaciated_areas
    • min_zoom
    • character encoding funk

Earlier scale sets:

  • ne_50m_rivers_lake_centerlines
    • min_zoom
  • ne_110m_rivers_lake_centerlines
    • min_zoom
  • ne_50m_minor_islands doesn't exist
  • ne_110m_minor_islands doesn't exist
  • ne_50m_reefs doesn't exist
  • ne_110m_reefs doesn't exist
  • ne_50m_geography_regions_points
    • min_zoom
    • character encoding funk?
  • ne_110m_geography_regions_points
    • min_zoom
    • character encoding funk? not an issue
  • ne_50m_geography_regions_elevation_points
    • min_zoom
    • character encoding funk? not an issue
  • ne_110m_geography_regions_elevation_points
    • min_zoom
    • character encoding funk? not an issue
  • ne_50m_geography_regions_polys
    • min_zoom
  • ne_110m_geography_regions_polys
    • min_zoom
  • ne_50m_geography_marine_polys
    • min_zoom
  • ne_110m_geography_marine_polys
    • min_zoom
    • fix char encoding :(
  • ne_50m_admin_0_disputed_areas
    • min_zoom
  • ne_110m_admin_0_disputed_areas doesn't exist
  • ne_50m_antarctic_ice_shelves_polys
    • min_zoom (and fix scalerank to 0)
    • character encoding funk no names
  • ne_110m_antarctic_ice_shelves_polys doesn't exist
  • ne_50m_glaciated_areas
    • min_zoom
    • character encoding funk no names
  • ne_110m_glaciated_areas
    • min_zoom
    • character encoding funk no names

Other:

  • Copy over adm0 property changes to adm0 Excel file
  • Copy over adm0 min_zoom changes to scale_rank label points
  • Copy over adm1 property changes from lakes file to prime file and to scale_rank label points
  • Copy over land min_zoom changes to scale_rank label points
  • Copy over ocean min_zoom changes to scale_rank label points
@nvkelso nvkelso changed the title Upgrades for web map display Upgrades for web map display (add min_zoom) Jul 8, 2017
@nvkelso
Copy link
Owner Author

nvkelso commented Jul 8, 2017

Subtask includes: #199

@nvkelso
Copy link
Owner Author

nvkelso commented Jul 30, 2017

This is still WIP after #199 merge for mostly land and ocean themes in 1:10m, 1:50m, and 1:110m themes.

@nvkelso
Copy link
Owner Author

nvkelso commented Aug 14, 2017

This remains the final outstanding task for the v4.0 milestone.

Once the min_zoom are copied over then the adm0 and adm1 themes will also need a rebuild using the scripts.

@nvkelso
Copy link
Owner Author

nvkelso commented Sep 28, 2017

Ocean scale rank is a little odd – We only have 22 big features that get constructed that are "negative space". We'd need a separate build for each zoom I think if we wanted to account for the min_zoom of the coastline, which is out of scope for this release. Workaround is to build the water off the WGS84 bounding box (so the map background is "blue") and then overlay the land which has proper positive space ranks.

@nvkelso
Copy link
Owner Author

nvkelso commented Sep 28, 2017

QA followups in #223.

@nvkelso nvkelso closed this as completed Sep 28, 2017
@nvkelso
Copy link
Owner Author

nvkelso commented Sep 28, 2017

(It's possible that additional themes should also have min_zoom values added – but that's for 4.1 milestone or later.)

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

No branches or pull requests

1 participant