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

park or protected area and urban area landuse kinds from NE needs normalized values #713

Closed
nvkelso opened this issue Apr 14, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@nvkelso
Copy link
Member

nvkelso commented Apr 14, 2016

These are all National Parks, they should get similar tags as the OSM features of the same class.

@nvkelso nvkelso modified the milestones: v0.10.0, v1.0.0 Apr 14, 2016
@nvkelso
Copy link
Member Author

nvkelso commented Apr 20, 2016

Moving to next release for load balancing.

@nvkelso nvkelso added the ready label May 9, 2016
@zerebubuth zerebubuth self-assigned this May 17, 2016
@zerebubuth
Copy link
Member

Could you give an example of this, please? I tried looking, but I couldn't find one, and it seems the low zoom landuse areas are coming from OSM anyway (apart from the urban area polygons). For example, in tile 5/5/12, I couldn't find any non-OSM ones:

$ curl -s http://vector.dev.mapzen.com/osm/landuse/5/5/12.json | \
  jq '.features[].properties.kind' | sort | uniq -c
    225 "national_park"
     51 "nature_reserve"
    831 "protected_area"
     23 "urban area"

@nvkelso nvkelso assigned nvkelso and unassigned zerebubuth May 18, 2016
@nvkelso nvkelso changed the title park or protected area landuse kind from NE needs normalized value park or protected area and urban area landuse kinds from NE needs normalized values Jun 6, 2016
@nvkelso
Copy link
Member Author

nvkelso commented Jun 6, 2016

Looks like urban area is still in v0.10 tiles, but I don't see park or protected area any more either. But it's not clear to me how the urban area features are coming thru looking at the queries.

The documentation in https://github.com/mapzen/vector-datasource/blob/master/docs/layers.md#landuse makes reference to both of these.

@zerebubuth
Copy link
Member

urban area is coming from this query in landuse. Changing what it's called should be a simple matter of changing the string there - I don't think we do any further processing to it.

@nvkelso
Copy link
Member Author

nvkelso commented Aug 10, 2016

Verified on dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants