{"action":"opened","issue":{"url":"https://api.github.com/repos/valhalla/valhalla/issues/1197","repository_url":"https://api.github.com/repos/valhalla/valhalla","labels_url":"https://api.github.com/repos/valhalla/valhalla/issues/1197/labels{/name}","comments_url":"https://api.github.com/repos/valhalla/valhalla/issues/1197/comments","events_url":"https://api.github.com/repos/valhalla/valhalla/issues/1197/events","html_url":"https://github.com/valhalla/valhalla/issues/1197","id":313129145,"number":1197,"title":"Issue with Speed = 0 assigned for a Ferry","user":{"login":"dnesbitt61","id":1838768,"avatar_url":"https://avatars1.githubusercontent.com/u/1838768?v=4","gravatar_id":"","url":"https://api.github.com/users/dnesbitt61","html_url":"https://github.com/dnesbitt61","followers_url":"https://api.github.com/users/dnesbitt61/followers","following_url":"https://api.github.com/users/dnesbitt61/following{/other_user}","gists_url":"https://api.github.com/users/dnesbitt61/gists{/gist_id}","starred_url":"https://api.github.com/users/dnesbitt61/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dnesbitt61/subscriptions","organizations_url":"https://api.github.com/users/dnesbitt61/orgs","repos_url":"https://api.github.com/users/dnesbitt61/repos","events_url":"https://api.github.com/users/dnesbitt61/events{/privacy}","received_events_url":"https://api.github.com/users/dnesbitt61/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"gknisely","id":2676277,"avatar_url":"https://avatars2.githubusercontent.com/u/2676277?v=4","gravatar_id":"","url":"https://api.github.com/users/gknisely","html_url":"https://github.com/gknisely","followers_url":"https://api.github.com/users/gknisely/followers","following_url":"https://api.github.com/users/gknisely/following{/other_user}","gists_url":"https://api.github.com/users/gknisely/gists{/gist_id}","starred_url":"https://api.github.com/users/gknisely/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gknisely/subscriptions","organizations_url":"https://api.github.com/users/gknisely/orgs","repos_url":"https://api.github.com/users/gknisely/repos","events_url":"https://api.github.com/users/gknisely/events{/privacy}","received_events_url":"https://api.github.com/users/gknisely/received_events","type":"User","site_admin":false},"assignees":[{"login":"gknisely","id":2676277,"avatar_url":"https://avatars2.githubusercontent.com/u/2676277?v=4","gravatar_id":"","url":"https://api.github.com/users/gknisely","html_url":"https://github.com/gknisely","followers_url":"https://api.github.com/users/gknisely/followers","following_url":"https://api.github.com/users/gknisely/following{/other_user}","gists_url":"https://api.github.com/users/gknisely/gists{/gist_id}","starred_url":"https://api.github.com/users/gknisely/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gknisely/subscriptions","organizations_url":"https://api.github.com/users/gknisely/orgs","repos_url":"https://api.github.com/users/gknisely/repos","events_url":"https://api.github.com/users/gknisely/events{/privacy}","received_events_url":"https://api.github.com/users/gknisely/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2018-04-11T00:01:02Z","updated_at":"2018-04-11T00:01:02Z","closed_at":null,"author_association":"OWNER","body":"http://localhost:8002/route?json={%22locations%22:[{%22lat%22:48.630367,%22lon%22:-2.0653240000000324},{%22lat%22:48.636323,%22lon%22:-2.0490159999999378},{%22lat%22:48.639159,%22lon%22:-2.036828000000014}],%22costing%22:%22bicycle%22,%22costing_options%22:{%22bicycle%22:{%22bicycle_type%22:%22Mountain%22,%22use_roads%22:0,%22use_hills%22:1,%22avoid_bad_surfaces%22:0,%22use_ferry%22:0}}}\r\n\r\nThis route has a location that snaps to a ferry that currently has speed = 0 assigned (perhaps due to duration logic). This creates a huge cost at that edge leading to the route taking an extremely long time to process (basically has to exhaust the rest of the network). A few possible fixes:\r\n1) make sure speed = 0 is not assigned (preferred), check the logic where speed is assigned based on duration.\r\n2) disallow edges with speed = 0 in the GetEdgeFilter lambdas within costing (desirable as well), \r\n3) disallow ferries for locations. The argument can be made that ferries should be allowed as the initial location (mobile app requesting a route while on the ferry) and for THROUGH locations so a route can be forced to take a ferry, so I don't think this is a good solution. \r\n\r\nThe last location in the request snaps to this way: https://www.openstreetmap.org/way/332454698#map=16/48.6404/-2.0371  - note that it has a duration so the logic that sets speed given duration likely needs to be modified and need to make sure it sets a speed of at least 1kph."}}
{"actor":{"display_login":"dnesbitt61"}}