Skip to content
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

StackingCVRegressor: multi_output option [#799] #802

Merged
merged 5 commits into from
Mar 6, 2021
Merged

StackingCVRegressor: multi_output option [#799] #802

merged 5 commits into from
Mar 6, 2021

Conversation

ChromaticIsobar
Copy link
Contributor

@ChromaticIsobar ChromaticIsobar commented Mar 6, 2021

Optionally allow multioutput regression in StackingCVRegressor:
Refers to issue #799

Fixes #799

ChromaticIsobar and others added 5 commits March 4, 2021 18:57
Optionally allow multioutput regression in StackingCVRegressor.
All code is as before, only X_y check is called with the extra
multi_output argument
@rasbt
Copy link
Owner

rasbt commented Mar 6, 2021

Looks great, thanks! Ported this behavior over to the StackingRegressor to make it consistent. Also updated the docs. Should be good to merge once the tests pass.

@rasbt rasbt merged commit fcd36b3 into rasbt:master Mar 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

StackingCVRegressor: multi_output option
2 participants