{"action":"created","issue":{"url":"https://api.github.com/repos/bjz/noise-rs/issues/111","repository_url":"https://api.github.com/repos/bjz/noise-rs","labels_url":"https://api.github.com/repos/bjz/noise-rs/issues/111/labels{/name}","comments_url":"https://api.github.com/repos/bjz/noise-rs/issues/111/comments","events_url":"https://api.github.com/repos/bjz/noise-rs/issues/111/events","html_url":"https://github.com/bjz/noise-rs/pull/111","id":162372593,"number":111,"title":"Optimize OpenSimplex 2D and 3D","user":{"login":"phoenixenero","id":9031092,"avatar_url":"https://avatars.githubusercontent.com/u/9031092?v=3","gravatar_id":"","url":"https://api.github.com/users/phoenixenero","html_url":"https://github.com/phoenixenero","followers_url":"https://api.github.com/users/phoenixenero/followers","following_url":"https://api.github.com/users/phoenixenero/following{/other_user}","gists_url":"https://api.github.com/users/phoenixenero/gists{/gist_id}","starred_url":"https://api.github.com/users/phoenixenero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phoenixenero/subscriptions","organizations_url":"https://api.github.com/users/phoenixenero/orgs","repos_url":"https://api.github.com/users/phoenixenero/repos","events_url":"https://api.github.com/users/phoenixenero/events{/privacy}","received_events_url":"https://api.github.com/users/phoenixenero/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-06-27T04:34:10Z","updated_at":"2016-06-27T12:52:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bjz/noise-rs/pulls/111","html_url":"https://github.com/bjz/noise-rs/pull/111","diff_url":"https://github.com/bjz/noise-rs/pull/111.diff","patch_url":"https://github.com/bjz/noise-rs/pull/111.patch"},"body":"Removes most of the duplicate calculations and tries to use previously-computed values as much as possible.\r\n\r\n### Benchmarks\r\n\r\n**Before**\r\n\r\n```\r\ntest bench_open_simplex2               ... bench:          26 ns/iter (+/- 1)\r\ntest bench_open_simplex2_64x64         ... bench:     104,816 ns/iter (+/- 1,131)\r\ntest bench_open_simplex3               ... bench:          52 ns/iter (+/- 3)\r\ntest bench_open_simplex3_64x64         ... bench:     169,494 ns/iter (+/- 794)\r\n```\r\n\r\n**After**\r\n\r\n```\r\ntest bench_open_simplex2               ... bench:          25 ns/iter (+/- 1)\r\ntest bench_open_simplex2_64x64         ... bench:      99,244 ns/iter (+/- 1,410)\r\ntest bench_open_simplex3               ... bench:          50 ns/iter (+/- 2)\r\ntest bench_open_simplex3_64x64         ... bench:     163,044 ns/iter (+/- 1,071)\r\n```\r\n\r\nRunning on Ubuntu 14.04 x64 in VirtualBox.\r\n\r\nRust toolchain:\r\n\r\n```\r\nnightly-x86_64-unknown-linux-gnu (default)\r\nrustc 1.11.0-nightly (ad7fe6521 2016-06-23)\r\n```"},"comment":{"url":"https://api.github.com/repos/bjz/noise-rs/issues/comments/228736698","html_url":"https://github.com/bjz/noise-rs/pull/111#issuecomment-228736698","issue_url":"https://api.github.com/repos/bjz/noise-rs/issues/111","id":228736698,"user":{"login":"bjz","id":695077,"avatar_url":"https://avatars.githubusercontent.com/u/695077?v=3","gravatar_id":"","url":"https://api.github.com/users/bjz","html_url":"https://github.com/bjz","followers_url":"https://api.github.com/users/bjz/followers","following_url":"https://api.github.com/users/bjz/following{/other_user}","gists_url":"https://api.github.com/users/bjz/gists{/gist_id}","starred_url":"https://api.github.com/users/bjz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bjz/subscriptions","organizations_url":"https://api.github.com/users/bjz/orgs","repos_url":"https://api.github.com/users/bjz/repos","events_url":"https://api.github.com/users/bjz/events{/privacy}","received_events_url":"https://api.github.com/users/bjz/received_events","type":"User","site_admin":false},"created_at":"2016-06-27T12:52:51Z","updated_at":"2016-06-27T12:52:51Z","body":"Nice finds!\r\n\r\nI'm guessing some of these things were in there to improve the understanding of the algorithm. Is there a way we could preserve some of that? Perhaps with comments? @Cifram or @amaranth might have some suggestions, seeing as they understand the algorithm more than me."}}
{"actor":{"display_login":"bjz"}}