nrenner / brouter-web

Web client for BRouter, a routing engine based on OpenStreetMap
https://brouter.de/brouter-web/
MIT License
363 stars 69 forks source link

New trekking profile suggests weird detour when `avoid_unsafe` is enabled #773

Open sjakobi opened 1 year ago

sjakobi commented 1 year ago

Example: https://brouter.de/brouter-web/#map=14/51.2280/12.0997/osm-mapnik-german_style,Waymarked_Trails-Cycling&lonlats=12.10475,51.217151;12.117341,51.240527

avoid_unsafe=false:

image

avoid_unsafe=true:

image

The problem IMHO is that the profile avoids this highway=unclassified, which has maxspeed=30 and hgv=agricultural. I don't think that road needs to be penalized so badly.

sjakobi commented 1 year ago

Note that Poutnik's classic Trekking profile doesn't do this. I wish there was a version of that profile enhanced to support the new environment tags.

The lack of support for smoothness tags is another issue with the new trekking and fastbike-lowtraffic profiles.

quaelnix commented 1 year ago

The penalty is so high because there is neither a cycle route nor a bicycle tag on this way.

Maybe use consider_traffic instead of avoid_unsafe in such cases?