{"action":"created","issue":{"url":"https://api.github.com/repos/koopjs/koop/issues/242","labels_url":"https://api.github.com/repos/koopjs/koop/issues/242/labels{/name}","comments_url":"https://api.github.com/repos/koopjs/koop/issues/242/comments","events_url":"https://api.github.com/repos/koopjs/koop/issues/242/events","html_url":"https://github.com/koopjs/koop/pull/242","id":104137796,"number":242,"title":"64% speedup in converting esri json","user":{"login":"dmfenton","id":7832202,"avatar_url":"https://avatars.githubusercontent.com/u/7832202?v=3","gravatar_id":"","url":"https://api.github.com/users/dmfenton","html_url":"https://github.com/dmfenton","followers_url":"https://api.github.com/users/dmfenton/followers","following_url":"https://api.github.com/users/dmfenton/following{/other_user}","gists_url":"https://api.github.com/users/dmfenton/gists{/gist_id}","starred_url":"https://api.github.com/users/dmfenton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmfenton/subscriptions","organizations_url":"https://api.github.com/users/dmfenton/orgs","repos_url":"https://api.github.com/users/dmfenton/repos","events_url":"https://api.github.com/users/dmfenton/events{/privacy}","received_events_url":"https://api.github.com/users/dmfenton/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"ngoldman","id":427322,"avatar_url":"https://avatars.githubusercontent.com/u/427322?v=3","gravatar_id":"","url":"https://api.github.com/users/ngoldman","html_url":"https://github.com/ngoldman","followers_url":"https://api.github.com/users/ngoldman/followers","following_url":"https://api.github.com/users/ngoldman/following{/other_user}","gists_url":"https://api.github.com/users/ngoldman/gists{/gist_id}","starred_url":"https://api.github.com/users/ngoldman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ngoldman/subscriptions","organizations_url":"https://api.github.com/users/ngoldman/orgs","repos_url":"https://api.github.com/users/ngoldman/repos","events_url":"https://api.github.com/users/ngoldman/events{/privacy}","received_events_url":"https://api.github.com/users/ngoldman/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2015-08-31T21:23:17Z","updated_at":"2015-08-31T21:43:23Z","closed_at":"2015-08-31T21:43:18Z","pull_request":{"url":"https://api.github.com/repos/koopjs/koop/pulls/242","html_url":"https://github.com/koopjs/koop/pull/242","diff_url":"https://github.com/koopjs/koop/pull/242.diff","patch_url":"https://github.com/koopjs/koop/pull/242.patch"},"body":"I went all functional on this and was able to achieve a 64% speedup without changing inputs or outputs. I suspect there is a lot more low hanging fruit like this throughout the lib.\r\n\r\nHere's the report from from Node-Bench\r\n\r\n```bash\r\nnode benchmark.js                                [16:57:31]\r\nbenchmarking /Users/.../repos/koop/benchmark.js\r\nPlease be patient.\r\n{ http_parser: '1.0',\r\n  node: '0.10.38',\r\n  v8: '3.14.5.9',\r\n  ares: '1.9.0-DEV',\r\n  uv: '0.10.36',\r\n  zlib: '1.2.8',\r\n  modules: '11',\r\n  openssl: '1.0.1m' }\r\nScores: (bigger is better)\r\n\r\nnew\r\nRaw:\r\n > 0.03290129611166501\r\n > 0.0323846908734053\r\n > 0.03260869565217391\r\n > 0.03235294117647059\r\nAverage (mean) 0.0325619059534287\r\n\r\nold\r\nRaw:\r\n > 0.011834319526627219\r\n > 0.011799410029498525\r\n > 0.011494252873563218\r\n > 0.011616650532429816\r\nAverage (mean) 0.011686158240529694\r\n\r\nWinner: new\r\nCompared with next highest (old), it's:\r\n64.11% faster\r\n2.79 times as fast\r\n0.45 order(s) of magnitude faster\r\nQUITE A BIT FASTER\r\n```\r\ncc @koopjs/dev "},"comment":{"url":"https://api.github.com/repos/koopjs/koop/issues/comments/136509759","html_url":"https://github.com/koopjs/koop/pull/242#issuecomment-136509759","issue_url":"https://api.github.com/repos/koopjs/koop/issues/242","id":136509759,"user":{"login":"ngoldman","id":427322,"avatar_url":"https://avatars.githubusercontent.com/u/427322?v=3","gravatar_id":"","url":"https://api.github.com/users/ngoldman","html_url":"https://github.com/ngoldman","followers_url":"https://api.github.com/users/ngoldman/followers","following_url":"https://api.github.com/users/ngoldman/following{/other_user}","gists_url":"https://api.github.com/users/ngoldman/gists{/gist_id}","starred_url":"https://api.github.com/users/ngoldman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ngoldman/subscriptions","organizations_url":"https://api.github.com/users/ngoldman/orgs","repos_url":"https://api.github.com/users/ngoldman/repos","events_url":"https://api.github.com/users/ngoldman/events{/privacy}","received_events_url":"https://api.github.com/users/ngoldman/received_events","type":"User","site_admin":false},"created_at":"2015-08-31T21:43:23Z","updated_at":"2015-08-31T21:43:23Z","body":":+1: "}}