8000 added example for BaggingRegressor by StephG123 · Pull Request #15219 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

added example for BaggingRegressor #15219

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

Closed
wants to merge 4 commits into from
Closed

Conversation

StephG123
Copy link

Toward #3846

This PR adds an example to ensemble.BaggingRegressor

@adrinjalali

@adrinjalali
Copy link
Member

@StephG123 would you have time to attend to the errors and fix them?

@StephG123
Copy link
Author
StephG123 commented Oct 15, 2019

@adrinjalali I'll do it once I'm back from holiday on Thursday.

work-acc-1 and others added 2 commits October 17, 2019 13:23
add random_seed for DecisionTreeRegressor; the output does not change

Co-Authored-By: Benjamin Bossan <BenjaminBossan@users.noreply.github.com>
@adrinjalali
Copy link
Member
adrinjalali commented Oct 17, 2019

You have pep8 errors and the CI is not running, have you branched from the latest master? You should merge upstream/master into your branch, as explained here: https://scikit-learn.org/dev/developers/contributing.html#contributing-code

@Malesche
Copy link
Contributor

@adrinjalali this should be closed because it was already resolved in #15196.

@adrinjalali
Copy link
Member

Yep, thanks for checking @Malesche

Sorry @StephG123 , it's hard to keep track of which ones are still open and which ones not. I hope you stick around with other contributions :)

@adrinjalali adrinjalali closed this Mar 2, 2020
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.

5 participants
0