{"action":"opened","issue":{"url":"https://api.github.com/repos/keepright/keepright/issues/61","repository_url":"https://api.github.com/repos/keepright/keepright","labels_url":"https://api.github.com/repos/keepright/keepright/issues/61/labels{/name}","comments_url":"https://api.github.com/repos/keepright/keepright/issues/61/comments","events_url":"https://api.github.com/repos/keepright/keepright/issues/61/events","html_url":"https://github.com/keepright/keepright/issues/61","id":176415780,"number":61,"title":"0270_motorways_connected_directly - extend query with vehicle=no; verify parking_aisle, services","user":{"login":"WK-GiHu","id":22146250,"avatar_url":"https://avatars.githubusercontent.com/u/22146250?v=3","gravatar_id":"","url":"https://api.github.com/users/WK-GiHu","html_url":"https://github.com/WK-GiHu","followers_url":"https://api.github.com/users/WK-GiHu/followers","following_url":"https://api.github.com/users/WK-GiHu/following{/other_user}","gists_url":"https://api.github.com/users/WK-GiHu/gists{/gist_id}","starred_url":"https://api.github.com/users/WK-GiHu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WK-GiHu/subscriptions","organizations_url":"https://api.github.com/users/WK-GiHu/orgs","repos_url":"https://api.github.com/users/WK-GiHu/repos","events_url":"https://api.github.com/users/WK-GiHu/events{/privacy}","received_events_url":"https://api.github.com/users/WK-GiHu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-09-12T16:05:33Z","updated_at":"2016-09-12T16:05:33Z","closed_at":null,"body":"I see error markers 'motorways_connected_directly' as false positives.\r\nThese connections are highway=service with vehicle=no.\r\nPlease verify if vehicle=no can be accepted as OK as \r\nthe outcome of default motorway access level + vehicle=no, equals to access=no.\r\n\r\nIn Line 136 the query use a exception if motorway connected to service=parking_aisle.\r\nA motorway should never connect to service=parking_aisle.\r\nService=parking_aisle should only connect to highway=service=driveway.\r\nPlease verify if you can remove service=parking_aisle. \r\n\r\nLine 128 includes \"'rest_area', \" but no \"'services', \"\r\nPlease verify if \"services', can be added.\r\n\r\nThink about to remove the query \"ways that come close (100 meters)\", \r\nbecause this will hide bad tagged connections.\r\n\r\nThink about to remove, <tag k=\"access\" v=\"emergency\"/>, \r\nthe Status are Abandoned (inactive).\r\n\r\nThe DIFF talking about:\r\n117,118c117,118\r\n< //\tthe other way has access=no|private OR\r\n< //\tthe other way is a service=parking_aisle\r\n---\r\n> //\tthe other way has access=no|private|vehicle=no OR\r\n> //\tthe other way is a highway=services|rest_area\r\n128c128\r\n< \t\t\tt.v NOT IN (... 'unclassified', 'rest_area', 'emergency_bay')\r\n---\r\n> \t\t\tt.v NOT IN (... 'unclassified', 'services', 'rest_area', 'emergency_bay')\r\n136c136\r\n< \t\t\t\t(t.k='service' AND t.v='parking_aisle'))\r\n---\r\n> \t\t\t\t (t.k='vehicle' AND t.v='no'))\r\n"}}
{"actor":{"display_login":"WK-GiHu"}}