{"action":"created","issue":{"url":"https://api.github.com/repos/opentripplanner/OpenTripPlanner/issues/1996","labels_url":"https://api.github.com/repos/opentripplanner/OpenTripPlanner/issues/1996/labels{/name}","comments_url":"https://api.github.com/repos/opentripplanner/OpenTripPlanner/issues/1996/comments","events_url":"https://api.github.com/repos/opentripplanner/OpenTripPlanner/issues/1996/events","html_url":"https://github.com/opentripplanner/OpenTripPlanner/issues/1996","id":89279524,"number":1996,"title":"Walk routing on trunk roads in UK","user":{"login":"marcusyoung","id":10391966,"avatar_url":"https://avatars.githubusercontent.com/u/10391966?v=3","gravatar_id":"","url":"https://api.github.com/users/marcusyoung","html_url":"https://github.com/marcusyoung","followers_url":"https://api.github.com/users/marcusyoung/followers","following_url":"https://api.github.com/users/marcusyoung/following{/other_user}","gists_url":"https://api.github.com/users/marcusyoung/gists{/gist_id}","starred_url":"https://api.github.com/users/marcusyoung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcusyoung/subscriptions","organizations_url":"https://api.github.com/users/marcusyoung/orgs","repos_url":"https://api.github.com/users/marcusyoung/repos","events_url":"https://api.github.com/users/marcusyoung/events{/privacy}","received_events_url":"https://api.github.com/users/marcusyoung/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-06-18T12:25:17Z","updated_at":"2015-06-19T10:19:32Z","closed_at":null,"body":"The attached screenshot shows a start and end point on the A483 in Tycroes, Wales. The A483 is a trunk road but here is going through a residential area and the speed limit is 30 mph and the road has pavements. But it seems that OTP thinks this road isn't suitable for a walk route, despite it having bus stops on it. I looked at this road in OSM and it had the following access tags (I have since updated OSM and foot and bicycle are now yes):\r\n\r\nAccess\r\n\r\nGeneral : yes\r\nFoot: unknown\r\nMotor Vehicles: yes\r\nBicycles: unknown\r\nHorses: unknown\r\n\r\nThe problem appears to be that OTP assumes that trunk roads are for driving only, from the following in DefaultWayPropertySetSource.java:\r\n\r\n        /* DRIVING ONLY */\r\n        setProperties(props, \"highway=trunk\", StreetTraversalPermission.CAR, 7.47, 7.47);\r\n        setProperties(props, \"highway=motorway\", StreetTraversalPermission.CAR, 8, 8);\r\n\r\nWhich I guess would be overridden if there is an access tag foot=yes . Trouble is, in the UK trunk roads do often pass through towns and cities, so the assumption of car only is incorrect. And anyway, there is no general legal restriction to walking or cycling on a trunk road in the UK. \r\n\r\nIdeally, this setting (and other routing rules such as average speeds of road types) would be configurable in build-config.json. That way it would be possible to tweak OTP routing rules to meet local requirements.\r\n\r\n![walk-route](https://cloud.githubusercontent.com/assets/10391966/8230975/f8df5bc2-15bc-11e5-8114-afe8a8651142.png)\r\nMarcus\r\n"},"comment":{"url":"https://api.github.com/repos/opentripplanner/OpenTripPlanner/issues/comments/113461215","html_url":"https://github.com/opentripplanner/OpenTripPlanner/issues/1996#issuecomment-113461215","issue_url":"https://api.github.com/repos/opentripplanner/OpenTripPlanner/issues/1996","id":113461215,"user":{"login":"abyrd","id":112871,"avatar_url":"https://avatars.githubusercontent.com/u/112871?v=3","gravatar_id":"","url":"https://api.github.com/users/abyrd","html_url":"https://github.com/abyrd","followers_url":"https://api.github.com/users/abyrd/followers","following_url":"https://api.github.com/users/abyrd/following{/other_user}","gists_url":"https://api.github.com/users/abyrd/gists{/gist_id}","starred_url":"https://api.github.com/users/abyrd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abyrd/subscriptions","organizations_url":"https://api.github.com/users/abyrd/orgs","repos_url":"https://api.github.com/users/abyrd/repos","events_url":"https://api.github.com/users/abyrd/events{/privacy}","received_events_url":"https://api.github.com/users/abyrd/received_events","type":"User","site_admin":false},"created_at":"2015-06-19T10:19:32Z","updated_at":"2015-06-19T10:19:32Z","body":"Hello,\n\nBeing able to specify all the rules in JSON would be nice, but it’s a fairly complicated change. A simpler solution is to make alternative subclasses that have additional rules (e.g. a UKWayPropertySetSource, though we might want to come up with a more pleasant name for these like UkWayTagInterpreter). Then we would just have a line in the JSON config to choose a PropertySetSource. The upside of this method is that the properties for each region would be checked into the OTP repository and available out of the box to other OTP users.\n\nWorking on the UKWayPropertySetSource as a Java file would not be as cumbersome as it might seem. If you work inside an IDE with a small OSM extract for a particular town, recompiling and restarting OTP to try out the configuration after each tweak is an automatic process and really not noticeably slower than changing JSON and restarting OTP on the command line without recompiling. Working on OTP this way in an IDE is much simpler than it used to be.\n\n-Andrew\n\n> On 19 Jun 2015, at 10:07, Marcus Young <notifications@github.com> wrote:\n> \n> Hi Marko \n> \n> Thanks. I realise that I can edit the relevant Java file but that then requires a build from master. This is very cumbersome if you are trying to tweak routing to local circumstances, with a rebuild after every change. So configuration via JSON would be great in the future. \n> \n> Marcus \n> \n> From: Marko Burjek \n> Sent: Friday, 19 June 2015 08:52 \n> To: opentripplanner/OpenTripPlanner \n> Reply To: opentripplanner/OpenTripPlanner \n> Cc: Young M.A. \n> Subject: {Disarmed} Re: [OpenTripPlanner] Walk routing on trunk roads in UK (#1996) \n> \n> \n> \n> It is configurable. You can change defaultWayPropertySetSource. It is also advisable to change default speed limits. \n> \n> Previously you could configure permissions in XML. But now only configuration in Java is possible. \n> But yes it would be easier to be able to configure this with JSON. \n> \n> And yes pedestrian permissions are added if foot=yes exists. \n> \n> — \n> Reply to this email directly or view it on GitHub<https://github.com/opentripplanner/OpenTripPlanner/issues/1996#issuecomment-113416937>. \n> \n> —\n> Reply to this email directly or view it on GitHub <https://github.com/opentripplanner/OpenTripPlanner/issues/1996#issuecomment-113424729>.\n> \n\n"}}