{"action":"created","issue":{"url":"https://api.github.com/repos/RcppCore/RcppParallel/issues/22","labels_url":"https://api.github.com/repos/RcppCore/RcppParallel/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/RcppCore/RcppParallel/issues/22/comments","events_url":"https://api.github.com/repos/RcppCore/RcppParallel/issues/22/events","html_url":"https://github.com/RcppCore/RcppParallel/pull/22","id":128919884,"number":22,"title":"bundle nt2 headers as well","user":{"login":"kevinushey","id":1976582,"avatar_url":"https://avatars.githubusercontent.com/u/1976582?v=3","gravatar_id":"","url":"https://api.github.com/users/kevinushey","html_url":"https://github.com/kevinushey","followers_url":"https://api.github.com/users/kevinushey/followers","following_url":"https://api.github.com/users/kevinushey/following{/other_user}","gists_url":"https://api.github.com/users/kevinushey/gists{/gist_id}","starred_url":"https://api.github.com/users/kevinushey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevinushey/subscriptions","organizations_url":"https://api.github.com/users/kevinushey/orgs","repos_url":"https://api.github.com/users/kevinushey/repos","events_url":"https://api.github.com/users/kevinushey/events{/privacy}","received_events_url":"https://api.github.com/users/kevinushey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-26T19:35:42Z","updated_at":"2016-01-26T20:51:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RcppCore/RcppParallel/pulls/22","html_url":"https://github.com/RcppCore/RcppParallel/pull/22","diff_url":"https://github.com/RcppCore/RcppParallel/pull/22.diff","patch_url":"https://github.com/RcppCore/RcppParallel/pull/22.patch"},"body":"This PR:\r\n\r\n- Adds a script that builds `nt2`, and copies over the generated headers -- this would hopefully be useful if we needed to rebuild `nt2` at a later date,\r\n- Amends `RcppParallel` headers where appropriate,\r\n- Adds a couple patches that clean up (bogus) compiler warnings in `nt2`.\r\n\r\n@jjallaire, let's chat before we merge -- we should probably change the header to `RCPP_PARALLEL_USE_NT2`, and we may want to ask that users explicitly include the headers they want from `nt2`, rather than just throwing everything in, as compiling a simple file now takes ~20 seconds, with `clang` gobbling up nearly a gigabyte of RAM in the process. (And this is on a 2015 MBP, so people with older systems would suffer a lot)\r\n\r\nI think we should also make sure we are prominently displaying that we are using `nt2`, and provide links back to the host repository when appropriate."},"comment":{"url":"https://api.github.com/repos/RcppCore/RcppParallel/issues/comments/175224162","html_url":"https://github.com/RcppCore/RcppParallel/pull/22#issuecomment-175224162","issue_url":"https://api.github.com/repos/RcppCore/RcppParallel/issues/22","id":175224162,"user":{"login":"eddelbuettel","id":673121,"avatar_url":"https://avatars.githubusercontent.com/u/673121?v=3","gravatar_id":"","url":"https://api.github.com/users/eddelbuettel","html_url":"https://github.com/eddelbuettel","followers_url":"https://api.github.com/users/eddelbuettel/followers","following_url":"https://api.github.com/users/eddelbuettel/following{/other_user}","gists_url":"https://api.github.com/users/eddelbuettel/gists{/gist_id}","starred_url":"https://api.github.com/users/eddelbuettel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eddelbuettel/subscriptions","organizations_url":"https://api.github.com/users/eddelbuettel/orgs","repos_url":"https://api.github.com/users/eddelbuettel/repos","events_url":"https://api.github.com/users/eddelbuettel/events{/privacy}","received_events_url":"https://api.github.com/users/eddelbuettel/received_events","type":"User","site_admin":false},"created_at":"2016-01-26T20:51:39Z","updated_at":"2016-01-26T20:51:39Z","body":"Wowser:  _Files changes: 9006_"}}