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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

added example for BaggingRegressor #15219

wants to merge 4 commits into from

Conversation

StephG123
Copy link

Toward #3846

This PR adds an example to ensemble.BaggingRegressor

@adrinjalali

BenjaminBossan
BenjaminBossan reviewed Oct 13, 2019
View reviewed changes
@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 •
edited
Loading

@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
@work-acc-1
add random seed for DecisionTreeRegressor, check output stays same
a8c5ada
@StephG123 @BenjaminBossan
Update sklearn/ensemble/bagging.py
0865143 
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 •
edited
Loading

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
Reviewers

@BenjaminBossan BenjaminBossan BenjaminBossan left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@StephG123 @adrinjalali @Malesche @BenjaminBossan @work-acc-1
0