{"action":"created","issue":{"url":"https://api.github.com/repos/topepo/C5.0/issues/2","labels_url":"https://api.github.com/repos/topepo/C5.0/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/topepo/C5.0/issues/2/comments","events_url":"https://api.github.com/repos/topepo/C5.0/issues/2/events","html_url":"https://github.com/topepo/C5.0/pull/2","id":85713782,"number":2,"title":"Added a module to prune boosted ensemble.","user":{"login":"NavamiK","id":12094191,"avatar_url":"https://avatars.githubusercontent.com/u/12094191?v=3","gravatar_id":"","url":"https://api.github.com/users/NavamiK","html_url":"https://github.com/NavamiK","followers_url":"https://api.github.com/users/NavamiK/followers","following_url":"https://api.github.com/users/NavamiK/following{/other_user}","gists_url":"https://api.github.com/users/NavamiK/gists{/gist_id}","starred_url":"https://api.github.com/users/NavamiK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NavamiK/subscriptions","organizations_url":"https://api.github.com/users/NavamiK/orgs","repos_url":"https://api.github.com/users/NavamiK/repos","events_url":"https://api.github.com/users/NavamiK/events{/privacy}","received_events_url":"https://api.github.com/users/NavamiK/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-06-06T06:56:34Z","updated_at":"2015-11-08T23:57:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/topepo/C5.0/pulls/2","html_url":"https://github.com/topepo/C5.0/pull/2","diff_url":"https://github.com/topepo/C5.0/pull/2.diff","patch_url":"https://github.com/topepo/C5.0/pull/2.patch"},"body":"A module to prune the boosted ensemble using kappa pruning algorithm to obtain user defined number of decision trees/rules."},"comment":{"url":"https://api.github.com/repos/topepo/C5.0/issues/comments/154887800","html_url":"https://github.com/topepo/C5.0/pull/2#issuecomment-154887800","issue_url":"https://api.github.com/repos/topepo/C5.0/issues/2","id":154887800,"user":{"login":"topepo","id":5731043,"avatar_url":"https://avatars.githubusercontent.com/u/5731043?v=3","gravatar_id":"","url":"https://api.github.com/users/topepo","html_url":"https://github.com/topepo","followers_url":"https://api.github.com/users/topepo/followers","following_url":"https://api.github.com/users/topepo/following{/other_user}","gists_url":"https://api.github.com/users/topepo/gists{/gist_id}","starred_url":"https://api.github.com/users/topepo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/topepo/subscriptions","organizations_url":"https://api.github.com/users/topepo/orgs","repos_url":"https://api.github.com/users/topepo/repos","events_url":"https://api.github.com/users/topepo/events{/privacy}","received_events_url":"https://api.github.com/users/topepo/received_events","type":"User","site_admin":false},"created_at":"2015-11-08T23:57:03Z","updated_at":"2015-11-08T23:57:03Z","body":"Navami ,\r\n\r\nThanks for the follow-up. No big deal about timing. A few things though: \r\n\r\nYour description in this thread is helpful, but you need to document it in the man file (or create a vingette with the details). You should also word it for non-techinical experts. \r\n\r\nPlease sync before submitting the pull request. This one would remove funcitonality from the package (including contributors). \r\n\r\nPlease remove these files in the PR:\r\n\r\n  * pkg/C50_0.1.0-21.tar.gz\r\n  * pkg/C50/.Rproj.user/\r\n  * pkg/C50/.RData\r\n  * pkg/C50.zip\r\n  * pkg/C50/.Rbuildignore\r\n  * pkg/C50/.Rhistory\r\n\r\nAlso, please make some test cases with the current version of the package and test that you get the same results when the additional functionality is not used. \r\n\r\nI'm very interested in these changes but, to be honest, the PR is really sloppy and it makes be hesitant about merging it in    \r\n"}}