{"action":"created","issue":{"url":"https://api.github.com/repos/memononen/recastnavigation/issues/97","labels_url":"https://api.github.com/repos/memononen/recastnavigation/issues/97/labels{/name}","comments_url":"https://api.github.com/repos/memononen/recastnavigation/issues/97/comments","events_url":"https://api.github.com/repos/memononen/recastnavigation/issues/97/events","html_url":"https://github.com/memononen/recastnavigation/pull/97","id":75263892,"number":97,"title":"Fix for wrongly simplified neighbour contours","user":{"login":"carlesros","id":12391765,"avatar_url":"https://avatars.githubusercontent.com/u/12391765?v=3","gravatar_id":"","url":"https://api.github.com/users/carlesros","html_url":"https://github.com/carlesros","followers_url":"https://api.github.com/users/carlesros/followers","following_url":"https://api.github.com/users/carlesros/following{/other_user}","gists_url":"https://api.github.com/users/carlesros/gists{/gist_id}","starred_url":"https://api.github.com/users/carlesros/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/carlesros/subscriptions","organizations_url":"https://api.github.com/users/carlesros/orgs","repos_url":"https://api.github.com/users/carlesros/repos","events_url":"https://api.github.com/users/carlesros/events{/privacy}","received_events_url":"https://api.github.com/users/carlesros/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-11T15:40:19Z","updated_at":"2015-08-15T12:26:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/memononen/recastnavigation/pulls/97","html_url":"https://github.com/memononen/recastnavigation/pull/97","diff_url":"https://github.com/memononen/recastnavigation/pull/97.diff","patch_url":"https://github.com/memononen/recastnavigation/pull/97.patch"},"body":"Two fixes for walkContour/simplifyContour:\r\n- Fix for floating error innacuracy when calculating the maximum\r\ndeviation during simplifyContour: ensure points are passed to\r\ndistancePtSeg on the same order for both neighbour contours.\r\n- Fix for regions touching only in one vertex: during walkContour mark\r\nvertices that touch another region in only one vertex, and then let\r\nsimplifyContour insert that vertex in the initial polygon.\r\n\r\nSee the post \"Two bugs in contour generation (and fixes)\" in Recast\r\nforum for a detailed description\r\nhttps://groups.google.com/forum/#!topic/recastnavigation/OHK5JWLvWs0"},"comment":{"url":"https://api.github.com/repos/memononen/recastnavigation/issues/comments/131366304","html_url":"https://github.com/memononen/recastnavigation/pull/97#issuecomment-131366304","issue_url":"https://api.github.com/repos/memononen/recastnavigation/issues/97","id":131366304,"user":{"login":"Sandern","id":1249842,"avatar_url":"https://avatars.githubusercontent.com/u/1249842?v=3","gravatar_id":"","url":"https://api.github.com/users/Sandern","html_url":"https://github.com/Sandern","followers_url":"https://api.github.com/users/Sandern/followers","following_url":"https://api.github.com/users/Sandern/following{/other_user}","gists_url":"https://api.github.com/users/Sandern/gists{/gist_id}","starred_url":"https://api.github.com/users/Sandern/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sandern/subscriptions","organizations_url":"https://api.github.com/users/Sandern/orgs","repos_url":"https://api.github.com/users/Sandern/repos","events_url":"https://api.github.com/users/Sandern/events{/privacy}","received_events_url":"https://api.github.com/users/Sandern/received_events","type":"User","site_admin":false},"created_at":"2015-08-15T12:26:01Z","updated_at":"2015-08-15T12:26:01Z","body":"It seems the changes assume dtTempContour::verts are unsigned shorts, rather than unsigned chars. Mistake, or am I missing something? "}}