[GraphHopper] Bike way type conversion

Stuart Adam engaric at hotmail.com
Tue Jun 2 15:29:56 UTC 2015


Hello ratrun

I can understand the usage of a small amount of flags however I would have thought logic more akin to the following.  This would only highlight something as a cycleway if it is intentionally marked as such rather than merely having a right of way.

Edited in the email so apologies for any formatting issues.

if (way.hasTag("bicycle", intendedValues))
{
	if(isPushingSection)
		wayType = WayType.OTHER_SMALL_WAY
	else if (“cycleway”.equals(highway)) 
		wayType = WayType.CYCLEWAY;
	else if (way.hasTag(“bicycle”,”designated) 
    		wayType = WayType.CYCLEWAY;
    	else if (roadValues.contains(highway))
		wayType = WayType.ROAD;
	

Any thoughts?

Sincerely
Stuart Adam


On 2 Jun 2015, at 16:10, ratrun <ratrun at gmx.at> wrote:

> Hello Stuart,
> 
> the waytype information is just used for the routing instructions such that a cycle rider gets a better idea what kind of way to look at. As the bits are limited and a more exact infomration is superflous for that purpose, I intentionally mangled all kind of ways somehow marked for bicycle usage together into "CYCLEWAY".
> 
> I also had some code which produced a statistic of the tour. It calcualted the distances per each paved and unpaved waytype. This code didn't make it into the master, although I think that it is an important feature for bicycle routing. The problem was that the code was too bicycle specific and required changes in all other flag encoders as well - see issue #209.
> 
> ratrun
> 
> Am 02.06.2015 um 16:17 schrieb Stuart Adam:
>> Hello all
>> 
>> I am starting to look at bike routing and I note in the handleBikeRelated method in BikeCommonFlagEncoder the following logic applies.
>> 
>>  if (way.hasTag("bicycle", intendedValues))
>>         {
>>             if (isPusingSection && !way.hasTag("bicycle", "designated"))
>>                 wayType = WayType.OTHER_SMALL_WAY;
>>             else
>>                 wayType = WayType.CYCLEWAY;
>>         } else if ("cycleway".equals(highway))
>>             wayType = WayType.CYCLEWAY;
>>         else if (roadValues.contains(highway))
>>             wayType = WayType.ROAD;
>> 
>> 
>> This does not seem correct to me as from my understanding this is taking the fact that a way has been marked as having a right of way for bicycles (but not a pushing section) then it is a full blown cyclepath.  In my mind at least cycleway implies dedicated and marked (normally with differently coloured tarmac) sections which is a much stronger indication for cycle use than just a bicycle right of way.
>> 
>> 
>> Am I correct and if so should this be changed or was there a reason for this decision in Graphhopper.
>> 
>> Sincerely
>> Stuart Adam
>> 
>> 
>> _______________________________________________
>> GraphHopper mailing list
>> GraphHopper at openstreetmap.org
>> https://lists.openstreetmap.org/listinfo/graphhopper
> 
> _______________________________________________
> GraphHopper mailing list
> GraphHopper at openstreetmap.org
> https://lists.openstreetmap.org/listinfo/graphhopper

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/graphhopper/attachments/20150602/eb3d32a3/attachment.html>


More information about the GraphHopper mailing list