{"action":"created","issue":{"url":"https://api.github.com/repos/patperry/r-mbest/issues/1","repository_url":"https://api.github.com/repos/patperry/r-mbest","labels_url":"https://api.github.com/repos/patperry/r-mbest/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/patperry/r-mbest/issues/1/comments","events_url":"https://api.github.com/repos/patperry/r-mbest/issues/1/events","html_url":"https://github.com/patperry/r-mbest/pull/1","id":138719325,"number":1,"title":"High performance modifications","user":{"login":"hacktuarial","id":9579628,"avatar_url":"https://avatars.githubusercontent.com/u/9579628?v=3","gravatar_id":"","url":"https://api.github.com/users/hacktuarial","html_url":"https://github.com/hacktuarial","followers_url":"https://api.github.com/users/hacktuarial/followers","following_url":"https://api.github.com/users/hacktuarial/following{/other_user}","gists_url":"https://api.github.com/users/hacktuarial/gists{/gist_id}","starred_url":"https://api.github.com/users/hacktuarial/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hacktuarial/subscriptions","organizations_url":"https://api.github.com/users/hacktuarial/orgs","repos_url":"https://api.github.com/users/hacktuarial/repos","events_url":"https://api.github.com/users/hacktuarial/events{/privacy}","received_events_url":"https://api.github.com/users/hacktuarial/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2016-03-05T20:44:03Z","updated_at":"2016-03-07T18:05:04Z","closed_at":"2016-03-05T21:42:06Z","pull_request":{"url":"https://api.github.com/repos/patperry/r-mbest/pulls/1","html_url":"https://github.com/patperry/r-mbest/pull/1","diff_url":"https://github.com/patperry/r-mbest/pull/1.diff","patch_url":"https://github.com/patperry/r-mbest/pull/1.patch"},"body":"This PR has two major modifications to the original mbest implementation.\r\n\r\n1. Fitting group-specific models in parallel using the _foreach_ package.\r\n2. Rewriting the observation-group index mapping in C++\r\n\r\nI had some difficulty incorporating C++ into the package, and ultimately used Roxygen to get it done. Consequently I had to move much of the NAMESPACE import/export functionality into Roxygen format. Unfortunately, this destroyed the ability to conditionally import the _sigma_ function from _lme4_ or _stats_ depending on the R version. "},"comment":{"url":"https://api.github.com/repos/patperry/r-mbest/issues/comments/193373513","html_url":"https://github.com/patperry/r-mbest/pull/1#issuecomment-193373513","issue_url":"https://api.github.com/repos/patperry/r-mbest/issues/1","id":193373513,"user":{"login":"patperry","id":43450,"avatar_url":"https://avatars.githubusercontent.com/u/43450?v=3","gravatar_id":"","url":"https://api.github.com/users/patperry","html_url":"https://github.com/patperry","followers_url":"https://api.github.com/users/patperry/followers","following_url":"https://api.github.com/users/patperry/following{/other_user}","gists_url":"https://api.github.com/users/patperry/gists{/gist_id}","starred_url":"https://api.github.com/users/patperry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/patperry/subscriptions","organizations_url":"https://api.github.com/users/patperry/orgs","repos_url":"https://api.github.com/users/patperry/repos","events_url":"https://api.github.com/users/patperry/events{/privacy}","received_events_url":"https://api.github.com/users/patperry/received_events","type":"User","site_admin":false},"created_at":"2016-03-07T18:05:04Z","updated_at":"2016-03-07T18:05:04Z","body":"FYI, `ranef.mhglm` is still using sequential code (which means that `predict.mhglm` is also sequential). Up for adding parallel support to these functions? To make this easier, I changed `mhglm` to return the `control` object as part of its result, so you can test `object$control$parallel` in these functions."}}