ratrun

Thanks for pointing me to this PR. Speed for paving_stones is not identical to paved speed, you are really a bit slower, so I would suggest 14 km/h as a...

> The raise was more meant to have a more realistic speed of bikes on footways as the maximum we can allow is the legal limit of 6km/h, but most...

> But unpaved also has 14 km/h and compacted and pebblestone even have 16km/h. Good catch. Maybe set `unpaved` to 12 like `gravel` and `compacted` and `pebbelstone` also to 14?...

Looks good! 😄

Closing now, because the PR would add to much complexity, see discussion in #2516.

Currently we are treating `access=no` and `vehicle=no` the same way with allowing this segment for bicycles. But according to https://wiki.openstreetmap.org/wiki/Tag:access%3Dno the intention of `access=no` is stronger. While on a segment...

> And should we remove the special case way.hasTag("highway", "cycleway")) I extended the check there in my first version of the PR, see #2981 > And then consider the "dismount"...

> but did you try it and check the results? How does it affect the quality of the landuse encoded value and the performance of the import? > Did you...

I ran a benchmark with the austria latest OSM file and compared your implementation against mine: Your version: `[main] INFO org.eclipse.jetty.server.Server - Started @330185ms` My version: `[main] INFO org.eclipse.jetty.server.Server -...

Thanks for all the feedback! In [f4ba483](https://github.com/graphhopper/graphhopper/pull/2812/commits/f4ba483202746f52ed00721a48921cb2da580419) I implemented the usage of the BBox and generate the polygon from the searchEnv. The result of a new benchmark was `[main] INFO...