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

No height property generated for building:levels #806

Closed
bcamper opened this issue May 10, 2016 · 3 comments
Closed

No height property generated for building:levels #806

bcamper opened this issue May 10, 2016 · 3 comments
Assignees
Milestone

Comments

@bcamper
Copy link
Collaborator

bcamper commented May 10, 2016

  • What did you see?
    building:levels tag in OSM did not result in a height property being generated in tile.
  • What did you expect to see?
    My understanding was that we were synthesizing a height property in tiles (where one doesn't already exist) from other data including building:levels.
  • What map location are you having problems with? City and country are helpful, as well as tile coordinates or latitude / longitude.

http://tangrams.github.io/bubble-wrap/#17/37.25730/127.05379
https://www.openstreetmap.org/way/161390790#map=18/37.25823/127.05604
https://vector.dev.mapzen.com/osm/all/16/55897/25449.topojson?api_key=vector-tiles-HqUVidw

buildings: {
   type: "GeometryCollection",
   geometries: [
      {
         type: "Polygon",
         properties: {
            building_levels: "25",
            kind: "Samsung tower 3",
            name: "R5",
            area: 46363,
            sort_key: 475,
            landuse_kind: "industrial",
            source: "openstreetmap.org",
            min_zoom: 13,
            id: 161390790
         },
      ...
@bcamper
Copy link
Collaborator Author

bcamper commented May 10, 2016

@zerebubuth
Copy link
Member

Ooops, thanks for pointing that out. Looks like we dropped the : earlier and didn't update downstream transforms?

@nvkelso nvkelso modified the milestones: v1.0.0, v0.10.2 May 10, 2016
@nvkelso
Copy link
Member

nvkelso commented May 10, 2016

Ooops, this is a regression over v0.9 tiles. Moving to new v0.10.2 bug fix release.

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

4 participants