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

Landuse layer min_zooms too detailed and prevents merging #2046

Open
nvkelso opened this issue Jan 11, 2022 · 0 comments
Open

Landuse layer min_zooms too detailed and prevents merging #2046

nvkelso opened this issue Jan 11, 2022 · 0 comments
Milestone

Comments

@nvkelso
Copy link
Member

nvkelso commented Jan 11, 2022

To encourage more feature merging, the min_zoom of features in the landuse layer should always be ints not floats. See 512px sized 13/1705/3108.mvt and 14/3412/6216.mvt where the grass has a tenths place on it's float that prevents feature merging.

Aside: These will also move to a new layer in #2025

Not all of these have float min_zoom now, but all of them should have int min_zoom after.

  • natural: wetland (wetland)
  • natural: mud (wetland) - this seems odd, isn't is mostly under water?
  • landuse: forest – but not all the managed "National Forests" (forest)
  • natural: forest – but not all the managed "National Forests" (forest)
  • landuse: wood – but not all the managed "National Forests" (forest)
  • natural: wood – but not all the managed "National Forests" (forest)
  • natural: scrub (scrub) – this is mapped to grassland now, but that's in error
  • natural: heath (scrub) – this is mapped to grassland now, but that's in error
  • landuse: grass (grassland)
  • natural: grassland (grassland)
  • landuse: meadow (grassland) - these sometimes have names
  • natural: desert (desert)
  • natural: bare_rock (desert) - should this be barren?
  • natural: sand (desert) - should this be barren?
  • natural: shingle (barren)
  • natural: scree (barren)
  • natural: stone (barren)
  • natural: rock (barren)
  • natural: glacier (glacier)
  • landuse: farm (farmland)
  • landuse: farmland (farmland)
  • landuse: rural (rural)
  • natural: tree_row
  • barrier: hedge (there may be special handling for barrier lines to bring over?)
@nvkelso nvkelso added this to the v2.0.0 milestone Jan 11, 2022
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

1 participant