{"action":"created","issue":{"url":"https://api.github.com/repos/hail-is/hail/issues/3415","repository_url":"https://api.github.com/repos/hail-is/hail","labels_url":"https://api.github.com/repos/hail-is/hail/issues/3415/labels{/name}","comments_url":"https://api.github.com/repos/hail-is/hail/issues/3415/comments","events_url":"https://api.github.com/repos/hail-is/hail/issues/3415/events","html_url":"https://github.com/hail-is/hail/pull/3415","id":316350129,"number":3415,"title":"minor decode optimization","user":{"login":"cseed","id":1244990,"avatar_url":"https://avatars3.githubusercontent.com/u/1244990?v=4","gravatar_id":"","url":"https://api.github.com/users/cseed","html_url":"https://github.com/cseed","followers_url":"https://api.github.com/users/cseed/followers","following_url":"https://api.github.com/users/cseed/following{/other_user}","gists_url":"https://api.github.com/users/cseed/gists{/gist_id}","starred_url":"https://api.github.com/users/cseed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cseed/subscriptions","organizations_url":"https://api.github.com/users/cseed/orgs","repos_url":"https://api.github.com/users/cseed/repos","events_url":"https://api.github.com/users/cseed/events{/privacy}","received_events_url":"https://api.github.com/users/cseed/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"tpoterba","id":10562794,"avatar_url":"https://avatars2.githubusercontent.com/u/10562794?v=4","gravatar_id":"","url":"https://api.github.com/users/tpoterba","html_url":"https://github.com/tpoterba","followers_url":"https://api.github.com/users/tpoterba/followers","following_url":"https://api.github.com/users/tpoterba/following{/other_user}","gists_url":"https://api.github.com/users/tpoterba/gists{/gist_id}","starred_url":"https://api.github.com/users/tpoterba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tpoterba/subscriptions","organizations_url":"https://api.github.com/users/tpoterba/orgs","repos_url":"https://api.github.com/users/tpoterba/repos","events_url":"https://api.github.com/users/tpoterba/events{/privacy}","received_events_url":"https://api.github.com/users/tpoterba/received_events","type":"User","site_admin":false},"assignees":[{"login":"tpoterba","id":10562794,"avatar_url":"https://avatars2.githubusercontent.com/u/10562794?v=4","gravatar_id":"","url":"https://api.github.com/users/tpoterba","html_url":"https://github.com/tpoterba","followers_url":"https://api.github.com/users/tpoterba/followers","following_url":"https://api.github.com/users/tpoterba/following{/other_user}","gists_url":"https://api.github.com/users/tpoterba/gists{/gist_id}","starred_url":"https://api.github.com/users/tpoterba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tpoterba/subscriptions","organizations_url":"https://api.github.com/users/tpoterba/orgs","repos_url":"https://api.github.com/users/tpoterba/repos","events_url":"https://api.github.com/users/tpoterba/events{/privacy}","received_events_url":"https://api.github.com/users/tpoterba/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2018-04-20T17:27:02Z","updated_at":"2018-04-20T19:48:28Z","closed_at":"2018-04-20T19:47:36Z","author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/hail-is/hail/pulls/3415","html_url":"https://github.com/hail-is/hail/pull/3415","diff_url":"https://github.com/hail-is/hail/pull/3415.diff","patch_url":"https://github.com/hail-is/hail/pull/3415.patch"},"body":"Only fast-path required arrays (e.g. in AD, PL).  Gives minor (7%) decode speedup on a shard of gnomad (51.5s => 47.5s):\r\n\r\n```\r\nmt = hl.read_matrix_table('gnomad.mt')\r\nmt = mt.drop_cols()\r\nmt.write('freq.mt', overwrite=True)\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/hail-is/hail/issues/comments/383203086","html_url":"https://github.com/hail-is/hail/pull/3415#issuecomment-383203086","issue_url":"https://api.github.com/repos/hail-is/hail/issues/3415","id":383203086,"user":{"login":"tpoterba","id":10562794,"avatar_url":"https://avatars2.githubusercontent.com/u/10562794?v=4","gravatar_id":"","url":"https://api.github.com/users/tpoterba","html_url":"https://github.com/tpoterba","followers_url":"https://api.github.com/users/tpoterba/followers","following_url":"https://api.github.com/users/tpoterba/following{/other_user}","gists_url":"https://api.github.com/users/tpoterba/gists{/gist_id}","starred_url":"https://api.github.com/users/tpoterba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tpoterba/subscriptions","organizations_url":"https://api.github.com/users/tpoterba/orgs","repos_url":"https://api.github.com/users/tpoterba/repos","events_url":"https://api.github.com/users/tpoterba/events{/privacy}","received_events_url":"https://api.github.com/users/tpoterba/received_events","type":"User","site_admin":false},"created_at":"2018-04-20T19:48:28Z","updated_at":"2018-04-20T19:48:28Z","author_association":"COLLABORATOR","body":"our decoder is spending a ton of time looking up requiredness and other compile-time information. A generated decoder would be a huge improvement. We're waiting until we can do this in C++, right?"}}
{"actor":{"display_login":"tpoterba"}}