{"action":"closed","issue":{"url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/2730","repository_url":"https://api.github.com/repos/Project-OSRM/osrm-backend","labels_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/2730/labels{/name}","comments_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/2730/comments","events_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/2730/events","html_url":"https://github.com/Project-OSRM/osrm-backend/issues/2730","id":168894747,"number":2730,"title":"Assertions related to lanes on 5.3 and master","user":{"login":"daniel-j-h","id":527241,"avatar_url":"https://avatars.githubusercontent.com/u/527241?v=3","gravatar_id":"","url":"https://api.github.com/users/daniel-j-h","html_url":"https://github.com/daniel-j-h","followers_url":"https://api.github.com/users/daniel-j-h/followers","following_url":"https://api.github.com/users/daniel-j-h/following{/other_user}","gists_url":"https://api.github.com/users/daniel-j-h/gists{/gist_id}","starred_url":"https://api.github.com/users/daniel-j-h/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniel-j-h/subscriptions","organizations_url":"https://api.github.com/users/daniel-j-h/orgs","repos_url":"https://api.github.com/users/daniel-j-h/repos","events_url":"https://api.github.com/users/daniel-j-h/events{/privacy}","received_events_url":"https://api.github.com/users/daniel-j-h/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Project-OSRM/osrm-backend/labels/Bug%20Report","name":"Bug Report","color":"e10c02"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/Project-OSRM/osrm-backend/milestones/14","html_url":"https://github.com/Project-OSRM/osrm-backend/milestones/5.3.0","labels_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/milestones/14/labels","id":1761228,"number":14,"title":"5.3.0","description":null,"creator":{"login":"TheMarex","id":606968,"avatar_url":"https://avatars.githubusercontent.com/u/606968?v=3","gravatar_id":"","url":"https://api.github.com/users/TheMarex","html_url":"https://github.com/TheMarex","followers_url":"https://api.github.com/users/TheMarex/followers","following_url":"https://api.github.com/users/TheMarex/following{/other_user}","gists_url":"https://api.github.com/users/TheMarex/gists{/gist_id}","starred_url":"https://api.github.com/users/TheMarex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TheMarex/subscriptions","organizations_url":"https://api.github.com/users/TheMarex/orgs","repos_url":"https://api.github.com/users/TheMarex/repos","events_url":"https://api.github.com/users/TheMarex/events{/privacy}","received_events_url":"https://api.github.com/users/TheMarex/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":20,"state":"open","created_at":"2016-05-11T22:35:19Z","updated_at":"2016-08-24T08:56:35Z","due_on":null,"closed_at":null},"comments":4,"created_at":"2016-08-02T14:08:47Z","updated_at":"2016-08-24T08:56:35Z","closed_at":"2016-08-24T08:56:35Z","body":"Splitting from https://github.com/Project-OSRM/osrm-backend/issues/2706: we're hitting the following lane assertion during extraction on the planet on master but also already on the 5.3 branch:\r\n\r\n```\r\n[assert] /mnt/ebs/osrm/2016-07-26-110507/src/extractor/guidance/turn_lane_augmentation.cpp:285\r\nin: osrm::extractor::guidance::lanes::LaneDataVector osrm::extractor::guidance::lanes::handleNoneValueAtSimpleTurn(osrm::extractor::guidance::lanes::LaneDataVector, const Intersection&): connection_count + 1 == lane_data.size()\r\nterminate called without an active exception\r\n```\r\n\r\nhttps://github.com/Project-OSRM/osrm-backend/blob/c7a1576100cb947af0e970644fb3f70dde63a9eb/src/extractor/guidance/turn_lane_augmentation.cpp#L283-L286\r\n\r\nI wrote up some code for printing the locations for the breaking intersections in the `fix/2706-debug` branch and deployed it. For the record, here are the first five:\r\n\r\n```\r\n>>> 2, 4\r\n Tags:\r\n\t1(none) from: 0 to: 0\r\n\t2(straight) from: 1 to: 1\r\n\t8(left) from: 2 to: 2\r\n\t256(uturn) from: 2 to: 2\r\n>>> Broken Intersection: \r\n(lon:7.828823, lat:47.976944)\r\n(lon:7.828583, lat:47.975312)\r\n(lon:7.828631, lat:47.97538)\r\n>>> 2, 4\r\n Tags:\r\n\t1(none) from: 0 to: 0\r\n\t2(straight) from: 1 to: 2\r\n\t8(left) from: 3 to: 3\r\n\t256(uturn) from: 3 to: 3\r\n>>> Broken Intersection: \r\n(lon:7.832026, lat:47.992681)\r\n(lon:7.832197, lat:47.992714)\r\n(lon:7.832428, lat:47.993222)\r\n(lon:7.83195, lat:47.992815)\r\n>>> 2, 4\r\n Tags:\r\n\t1(none) from: 0 to: 0\r\n\t2(straight) from: 1 to: 1\r\n\t8(left) from: 2 to: 2\r\n\t256(uturn) from: 2 to: 2\r\n>>> Broken Intersection: \r\n(lon:7.809372, lat:48.011799)\r\n(lon:7.809382, lat:48.011674)\r\n(lon:7.809655, lat:48.011597)\r\n(lon:7.809555, lat:48.011808)\r\n>>> 2, 4\r\n Tags:\r\n\t1(none) from: 0 to: 0\r\n\t2(straight) from: 1 to: 1\r\n\t8(left) from: 2 to: 3\r\n\t256(uturn) from: 3 to: 3\r\n>>> Broken Intersection: \r\n(lon:9.183196, lat:48.776604)\r\n(lon:9.182912, lat:48.776235)\r\n(lon:9.182994, lat:48.776181)\r\n>>> 2, 4\r\n Tags:\r\n\t1(none) from: 0 to: 1\r\n\t2(straight) from: 2 to: 2\r\n\t8(left) from: 2 to: 3\r\n\t256(uturn) from: 4 to: 4\r\n>>> Broken Intersection: \r\n(lon:-96.884376, lat:32.826898)\r\n(lon:-96.885507, lat:32.82768)\r\n(lon:-96.885885, lat:32.827015)\r\n```\r\n\r\nA small extract where we're hitting this would is:\r\nhttp://download.geofabrik.de/europe/germany/baden-wuerttemberg.html\r\n\r\n@danpat could reduce my Cucumber test case even further down to a single lane tag: https://github.com/Project-OSRM/osrm-backend/commit/57ca921dcdf4ef24f58b2c360c1264a98e4480c0\r\n"}}
{"actor":{"display_login":"MoKob"}}