-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
Add multi-output support to the bagging module #3449
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I'll try to work on this one. |
Thanks @ldirer |
Is this issue still open? If not, I'd like to pick it up! |
I haven't really worked on it so you are welcome to pick it up if you want! |
You might want to have a look at the forest code from 0.14 which is similar to what should be done in the multi-output setting. https://github.com/scikit-learn/scikit-learn/blob/0.14.X/sklearn/ensemble/forest.py |
Yup! I've been cribbing a good amount from the I think everything is just about done, but I do have a quick question: is there a notion of a "decision function" in a multi-output setting? I did a quick look and all the estimators implementing a |
With multi-output data, I would go with a list of arrays. One array for each output. |
Okay - I've submitted a pull request #3798 |
implemented in #9707 |
This would nice to add multi-output support to bagging as in the
sklearn.ensemble.forest
module. Some code could be refactored.The text was updated successfully, but these errors were encountered: