{"action":"created","issue":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/7118","repository_url":"https://api.github.com/repos/scikit-learn/scikit-learn","labels_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/7118/labels{/name}","comments_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/7118/comments","events_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/7118/events","html_url":"https://github.com/scikit-learn/scikit-learn/pull/7118","id":168400246,"number":7118,"title":"[MRG+1] Remove heavy memory footprint in BaseBagging due to OOB scoring","user":{"login":"staubda","id":6806542,"avatar_url":"https://avatars.githubusercontent.com/u/6806542?v=3","gravatar_id":"","url":"https://api.github.com/users/staubda","html_url":"https://github.com/staubda","followers_url":"https://api.github.com/users/staubda/followers","following_url":"https://api.github.com/users/staubda/following{/other_user}","gists_url":"https://api.github.com/users/staubda/gists{/gist_id}","starred_url":"https://api.github.com/users/staubda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/staubda/subscriptions","organizations_url":"https://api.github.com/users/staubda/orgs","repos_url":"https://api.github.com/users/staubda/repos","events_url":"https://api.github.com/users/staubda/events{/privacy}","received_events_url":"https://api.github.com/users/staubda/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/labels/Needs%20Review","name":"Needs Review","color":"f7c6c7"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":42,"created_at":"2016-07-29T19:47:50Z","updated_at":"2016-08-23T06:59:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/pulls/7118","html_url":"https://github.com/scikit-learn/scikit-learn/pull/7118","diff_url":"https://github.com/scikit-learn/scikit-learn/pull/7118.diff","patch_url":"https://github.com/scikit-learn/scikit-learn/pull/7118.patch"},"body":"Fixes #6877.\r\n\r\n- Remove `estimators_samples` attribute from `BaseBagging`. Replace with\r\n  `estimators_samples_` method wrapped with `@property` decorator. The \r\n  new method calls `_get_estimators_data_draws` described below.\r\n- Add '_seeds' attribute to `BaseBagging` to store seeds each estimator \r\n  is initialized with.\r\n- Add method `_get_estimators_data_draws` to `BaseBagging` that\r\n  returns a generator producing sample and feature indices (or masks if desired) \r\n  on demand.\r\n- Slight refactor of `_parallel_build_estimators()` to\r\n  isolate and group lines accessing random state.\r\n"},"comment":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/comments/241644210","html_url":"https://github.com/scikit-learn/scikit-learn/pull/7118#issuecomment-241644210","issue_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/7118","id":241644210,"user":{"login":"jnothman","id":78827,"avatar_url":"https://avatars.githubusercontent.com/u/78827?v=3","gravatar_id":"","url":"https://api.github.com/users/jnothman","html_url":"https://github.com/jnothman","followers_url":"https://api.github.com/users/jnothman/followers","following_url":"https://api.github.com/users/jnothman/following{/other_user}","gists_url":"https://api.github.com/users/jnothman/gists{/gist_id}","starred_url":"https://api.github.com/users/jnothman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jnothman/subscriptions","organizations_url":"https://api.github.com/users/jnothman/orgs","repos_url":"https://api.github.com/users/jnothman/repos","events_url":"https://api.github.com/users/jnothman/events{/privacy}","received_events_url":"https://api.github.com/users/jnothman/received_events","type":"User","site_admin":false},"created_at":"2016-08-23T06:59:10Z","updated_at":"2016-08-23T06:59:10Z","body":"Abracadabra..."}}
{"actor":{"display_login":"jnothman"}}