bicycle=yes and access=no

[emmexx]

Take a look at this route
http://map.project-osrm.org/OO

In the demo the speed profile is for cars but I get the same route when using a bicycle profile.
Via San Prospero has the following tags:
highway=residential
bicycle=yes
access=no

In real life I'd go straight from start to target, it's a 60-70 meters trip, not a 1.33 km one. :-(

In ExtractorCallBacks.h at lines 220-221 w.access is set to false and prevails over bicycle=yes, if I understand the code correctly.
OSM wiki says:
"bicycle yes Where bicycles are permitted, overriding default access..."
http://wiki.openstreetmap.org/wiki/Bicycle

If there's no work on this part of the code I'll try to patch it.

maxx

[lapinos03]

I encountered this issue too. So I patched it straight away :

            //Okay, do we have access to that way?
            if(0 < access.size()) { //fastest way to check for non-empty string
                //If access is forbidden, we don't want to route there.
                if((access == "no" /*lapinos03*//*|| access == "agricultural" || access == "forestry" || access == "delivery"*/)
                   /*lapinos03*/ && !("yes" == accessTag || "designated" == accessTag)
                   ) { //Todo: this is still hard coded
                    w.access = false;
                }
                if(settings.accessRestrictionKeys.find(access) != settings.accessRestrictionKeys.end()) {
                    w.isAccessRestricted = true;
                }
            }

(I also removed the tests on agricultural, forestry, etc... for my needs)

[emiltin]

this pull request might be of interest: emiltin@599c026

you can also try running the cucumber tests to see check what effects the change has. adding/modifying tests are also possible :-)

[DennisOSRM]

Will be fixed with implementation of issue #1. Therefore closing as duplicate.