{"action":"created","issue":{"url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/issues/20","labels_url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/issues/20/labels{/name}","comments_url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/issues/20/comments","events_url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/issues/20/events","html_url":"https://github.com/mapzen/mapbox-vector-tile/issues/20","id":113038012,"number":20,"title":"Update for multi-polygon compliance with spec v1.0.2","user":{"login":"bcamper","id":16733,"avatar_url":"https://avatars.githubusercontent.com/u/16733?v=3","gravatar_id":"","url":"https://api.github.com/users/bcamper","html_url":"https://github.com/bcamper","followers_url":"https://api.github.com/users/bcamper/followers","following_url":"https://api.github.com/users/bcamper/following{/other_user}","gists_url":"https://api.github.com/users/bcamper/gists{/gist_id}","starred_url":"https://api.github.com/users/bcamper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bcamper/subscriptions","organizations_url":"https://api.github.com/users/bcamper/orgs","repos_url":"https://api.github.com/users/bcamper/repos","events_url":"https://api.github.com/users/bcamper/events{/privacy}","received_events_url":"https://api.github.com/users/bcamper/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/labels/qa%20needed","name":"qa needed","color":"fbca04"},{"url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/labels/send%20to%20staging","name":"send to staging","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"rmarianski","id":53770,"avatar_url":"https://avatars.githubusercontent.com/u/53770?v=3","gravatar_id":"","url":"https://api.github.com/users/rmarianski","html_url":"https://github.com/rmarianski","followers_url":"https://api.github.com/users/rmarianski/followers","following_url":"https://api.github.com/users/rmarianski/following{/other_user}","gists_url":"https://api.github.com/users/rmarianski/gists{/gist_id}","starred_url":"https://api.github.com/users/rmarianski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmarianski/subscriptions","organizations_url":"https://api.github.com/users/rmarianski/orgs","repos_url":"https://api.github.com/users/rmarianski/repos","events_url":"https://api.github.com/users/rmarianski/events{/privacy}","received_events_url":"https://api.github.com/users/rmarianski/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/milestones/1","html_url":"https://github.com/mapzen/mapbox-vector-tile/milestones/v0.5.0","labels_url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/milestones/1/labels","id":1390472,"number":1,"title":"v0.5.0","description":"","creator":{"login":"nvkelso","id":853051,"avatar_url":"https://avatars.githubusercontent.com/u/853051?v=3","gravatar_id":"","url":"https://api.github.com/users/nvkelso","html_url":"https://github.com/nvkelso","followers_url":"https://api.github.com/users/nvkelso/followers","following_url":"https://api.github.com/users/nvkelso/following{/other_user}","gists_url":"https://api.github.com/users/nvkelso/gists{/gist_id}","starred_url":"https://api.github.com/users/nvkelso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nvkelso/subscriptions","organizations_url":"https://api.github.com/users/nvkelso/orgs","repos_url":"https://api.github.com/users/nvkelso/repos","events_url":"https://api.github.com/users/nvkelso/events{/privacy}","received_events_url":"https://api.github.com/users/nvkelso/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":1,"state":"open","created_at":"2015-11-03T05:53:06Z","updated_at":"2015-11-03T06:58:55Z","due_on":"2015-11-06T08:00:00Z","closed_at":null},"comments":10,"created_at":"2015-10-23T15:03:20Z","updated_at":"2015-11-12T21:47:17Z","closed_at":null,"body":"The upcoming spec v1.0.2 clarifies how multi-polygons should be encoded: all polygons should be \"flattened\" into one series of rings, with the winding order of the rings used to determine which are interior/exterior. An exterior ring indicates the start of a new polygon.\r\n\r\nI believe this differs from our current handling of multi-polygons, where we \"exploded\" each polygon into its own feature, and copied the properties of each (as there was no official handling of multi-polygons previously). We should update to be compliant. Tangram already handles the new multi-polygon encoding so no client changes are needed on that side (style rules should be compatible with either method).\r\n\r\nhttps://github.com/mapbox/vector-tile-spec/pull/36\r\n"},"comment":{"url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/issues/comments/156246276","html_url":"https://github.com/mapzen/mapbox-vector-tile/issues/20#issuecomment-156246276","issue_url":"https://api.github.com/repos/mapzen/mapbox-vector-tile/issues/20","id":156246276,"user":{"login":"bcamper","id":16733,"avatar_url":"https://avatars.githubusercontent.com/u/16733?v=3","gravatar_id":"","url":"https://api.github.com/users/bcamper","html_url":"https://github.com/bcamper","followers_url":"https://api.github.com/users/bcamper/followers","following_url":"https://api.github.com/users/bcamper/following{/other_user}","gists_url":"https://api.github.com/users/bcamper/gists{/gist_id}","starred_url":"https://api.github.com/users/bcamper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bcamper/subscriptions","organizations_url":"https://api.github.com/users/bcamper/orgs","repos_url":"https://api.github.com/users/bcamper/repos","events_url":"https://api.github.com/users/bcamper/events{/privacy}","received_events_url":"https://api.github.com/users/bcamper/received_events","type":"User","site_admin":false},"created_at":"2015-11-12T21:47:17Z","updated_at":"2015-11-12T21:47:17Z","body":"I was wrong, the Mapbox decoding lib doesn't provide this! This is fixed in Tangram here, when we are ready to merge:\r\n\r\nhttps://github.com/tangrams/tangram/compare/decode-multipolygon"}}