{"action":"opened","issue":{"url":"https://api.github.com/repos/mapbox/mapbox-gl-js/issues/7609","repository_url":"https://api.github.com/repos/mapbox/mapbox-gl-js","labels_url":"https://api.github.com/repos/mapbox/mapbox-gl-js/issues/7609/labels{/name}","comments_url":"https://api.github.com/repos/mapbox/mapbox-gl-js/issues/7609/comments","events_url":"https://api.github.com/repos/mapbox/mapbox-gl-js/issues/7609/events","html_url":"https://github.com/mapbox/mapbox-gl-js/issues/7609","id":383247030,"node_id":"MDU6SXNzdWUzODMyNDcwMzA=","number":7609,"title":"Don't use pauseable placement when fadeDuration is 0","user":{"login":"ChrisLoer","id":375121,"node_id":"MDQ6VXNlcjM3NTEyMQ==","avatar_url":"https://avatars2.githubusercontent.com/u/375121?v=4","gravatar_id":"","url":"https://api.github.com/users/ChrisLoer","html_url":"https://github.com/ChrisLoer","followers_url":"https://api.github.com/users/ChrisLoer/followers","following_url":"https://api.github.com/users/ChrisLoer/following{/other_user}","gists_url":"https://api.github.com/users/ChrisLoer/gists{/gist_id}","starred_url":"https://api.github.com/users/ChrisLoer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ChrisLoer/subscriptions","organizations_url":"https://api.github.com/users/ChrisLoer/orgs","repos_url":"https://api.github.com/users/ChrisLoer/repos","events_url":"https://api.github.com/users/ChrisLoer/events{/privacy}","received_events_url":"https://api.github.com/users/ChrisLoer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-11-21T18:35:18Z","updated_at":"2018-11-21T18:35:18Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"When the global `fadeDuration` option is set to `0`, we run collision detection on every frame, and short-circuit collision fade animations. This makes every frame significantly more expensive to render, but also is supposed to guarantee that when tiles load they have all their symbols fully drawn on the first frame.\r\n\r\nOur short-circuiting logic doesn't take account for the fact that we will split collision detection between multiple frames if it takes longer than 2ms. So when a new tile loads, we'll immediately start running collision detection, but if it doesn't finish within 2ms, we'll go ahead and render the tile for a frame and then finish collision detection on the next frame.\r\n\r\nI think we should just always do full placement when `fadeDuration === 0`. It's more expensive, but I think it's what people expect.\r\n\r\ncc @tristen @mzdraper @ansis "}}
{"actor":{"display_login":"ChrisLoer"}}