{"action":"created","issue":{"url":"https://api.github.com/repos/kanaka/noVNC/issues/488","labels_url":"https://api.github.com/repos/kanaka/noVNC/issues/488/labels{/name}","comments_url":"https://api.github.com/repos/kanaka/noVNC/issues/488/comments","events_url":"https://api.github.com/repos/kanaka/noVNC/issues/488/events","html_url":"https://github.com/kanaka/noVNC/pull/488","id":78705972,"number":488,"title":"Additional Performance Improvements","user":{"login":"DirectXMan12","id":119737,"avatar_url":"https://avatars.githubusercontent.com/u/119737?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectXMan12","html_url":"https://github.com/DirectXMan12","followers_url":"https://api.github.com/users/DirectXMan12/followers","following_url":"https://api.github.com/users/DirectXMan12/following{/other_user}","gists_url":"https://api.github.com/users/DirectXMan12/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectXMan12/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectXMan12/subscriptions","organizations_url":"https://api.github.com/users/DirectXMan12/orgs","repos_url":"https://api.github.com/users/DirectXMan12/repos","events_url":"https://api.github.com/users/DirectXMan12/events{/privacy}","received_events_url":"https://api.github.com/users/DirectXMan12/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kanaka/noVNC/labels/WIP","name":"WIP","color":"bfd4f2"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2015-05-20T21:30:32Z","updated_at":"2015-08-04T20:56:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kanaka/noVNC/pulls/488","html_url":"https://github.com/kanaka/noVNC/pull/488","diff_url":"https://github.com/kanaka/noVNC/pull/488.diff","patch_url":"https://github.com/kanaka/noVNC/pull/488.patch"},"body":"**WIP: Do Not Merge**\r\n\r\nThis commit contains a variety of additional performance improvements, currently focused around the tight indexed-to-rgb encoding.  It includes the commits from #487 \r\n\r\nCurrently, I've reduced the time a further 50% from #487 on the qemu ls test.\r\n\r\nThis makes use of Typed Arrays.  The big tradeoff here is that we preallocate, meaning noVNC starts off using more RAM (we preallocate 4 MiB for the receive queue, `4 * width * height` B for a work buffer, and 1 KiB for a palette buffer).\r\n\r\nCC @kanaka "},"comment":{"url":"https://api.github.com/repos/kanaka/noVNC/issues/comments/127759274","html_url":"https://github.com/kanaka/noVNC/pull/488#issuecomment-127759274","issue_url":"https://api.github.com/repos/kanaka/noVNC/issues/488","id":127759274,"user":{"login":"DirectXMan12","id":119737,"avatar_url":"https://avatars.githubusercontent.com/u/119737?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectXMan12","html_url":"https://github.com/DirectXMan12","followers_url":"https://api.github.com/users/DirectXMan12/followers","following_url":"https://api.github.com/users/DirectXMan12/following{/other_user}","gists_url":"https://api.github.com/users/DirectXMan12/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectXMan12/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectXMan12/subscriptions","organizations_url":"https://api.github.com/users/DirectXMan12/orgs","repos_url":"https://api.github.com/users/DirectXMan12/repos","events_url":"https://api.github.com/users/DirectXMan12/events{/privacy}","received_events_url":"https://api.github.com/users/DirectXMan12/received_events","type":"User","site_admin":false},"created_at":"2015-08-04T20:56:51Z","updated_at":"2015-08-04T20:56:51Z","body":"Alright, this update should address @kanaka's concerns (render queue support added, message functions take the websock object as a parameter, `mouse_arr` has been nixed) with the exception of render queue support for `finishTile`, since we'd need to have render queue support for all the tile functions, and I'm not sure what kind of performance impact that would have.  I can take a look, but it would be helpful if someone has a HEXTILE recording to test with.  Render queue support for the tile functions can be added in later pretty easily.\r\n\r\n@kanaka , @samhed, @astrand  : any objections to merging this (one Travis finishes up)?"}}