vector-datasource icon indicating copy to clipboard operation
vector-datasource copied to clipboard

Add fell kind to landuse

Open rwrx opened this issue 5 years ago • 5 comments

Add fell kind to landuse

rwrx avatar Sep 30 '19 10:09 rwrx

Are there also names for these, if so we need to add collision_rank.

nvkelso avatar Sep 30 '19 18:09 nvkelso

I am not sure which number for fell should I set. I have looked at bare_rock, desert, grassland, heath, sand, shingle and they does not have set sort_rank. I haven't found a fell with a name. Is there some online tool to do osm query?

rwrx avatar Oct 01 '19 11:10 rwrx

Looks like a few fell features do have names:

  • https://taginfo.openstreetmap.org/tags/natural=fell#combinations

sort_order:

  • https://github.com/tilezen/vector-datasource/blob/master/spreadsheets/sort_rank/landuse.csv
  • Insert where grass is now, so fell takes 35 and all the rest above starting with grass increment +1 so power_line becomes 275.
  • There will be other test failures associated with this change that you'd need to also update, see CircleCI for which ones after you push the changes.

collision_order:

  • https://github.com/tilezen/vector-datasource/blob/master/spreadsheets/collision_rank.yaml#L602
  • Much simplier to just inset above grass, and build system takes care of auto incrementing everything.
  • You may still have associated test failures in Circle that you'll update for after pushing, too.

nvkelso avatar Oct 07 '19 18:10 nvkelso

I have updated sort and collision ranks, but now Circle is complaining that there is no 265 sort rank, because I have incremented it to 266:

dam,LineString;MultiLineString,266

I am not sure how to change this Circle test.

rwrx avatar Oct 09 '19 10:10 rwrx

I have added test, but I am not sure into which file I should have put it in and I am not sure whether it is correctly written.

rwrx avatar Oct 09 '19 11:10 rwrx