GH #72: Don't hardcode bikes_allowed for County Connection

bikehopper / graphhopper

Open source routing engine for OpenStreetMap. Use it as Java library or standalone web server.

https://www.graphhopper.com/open-source/

Apache License 2.0

3 stars 0 forks source link

GH #72: Don't hardcode bikes_allowed for County Connection #144

Closed rsarathy closed 3 months ago

graue commented 3 months ago

What's your plan to either transform the GTFS feed or fix it upstream?

rsarathy commented 3 months ago

I don't know how to go about modifying the data in GTFS itself just yet.

I did make one discovery, which is that presently, all County Connection routes have bikes_allowed=1 set. This means we can remove that one piece of control flow in Trip.java. I'll limit this PR to that specific change.

FWIW we don't have a lot of bikes_allowed=2 in our GTFS data. There are 159,847 routes in total, of which only 8 have bikes_allowed=2 set.

rsarathy commented 3 months ago

Multimodal trip leveraging County Connection without the hardcoding in Trip.java.