10000 GradientBoostingRegressor always sets presort to auto · Issue #5857 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

GradientBoostingRegressor always sets presort to auto #5857

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
mcculloh opened this issue Nov 16, 2015 · 4 comments
Closed

GradientBoostingRegressor always sets presort to auto #5857

mcculloh opened this issue Nov 16, 2015 · 4 comments

Comments

@mcculloh
Copy link

In the call to super inside __init__() GradientBoostingRegressor does not pass thru the presort variable to the superclass:

    super(GradientBoostingRegressor, self).__init__(
        loss=loss, learning_rate=learning_rate, n_estimators=n_estimators,
        ...
        max_leaf_nodes=max_leaf_nodes, warm_start=warm_start,
        presort='auto')
@mcculloh mcculloh mentioned this issue Nov 16, 2015
Merged
@amueller
Copy link
Member

ping @jmschrei @glouppe @arjoly
again ;)

@jmschrei
Copy link
Member

I'm suspecting that that PR introduced both of the issues. I think there is a check_X_y call which is causing the other issue which I missed since I was using smaller datasets. I've been swamped with a few presentations I have to do this week, but I'll be available to take a look at both soon.

@mcculloh
Copy link
Author

I added a second commit. It looks like the logic to activate the presort would only actually work if you explicitly set the parameter to 'True' . Since the super() call clobbers this, I don't know that it ever actually got turned on.

@glouppe
Copy link
Contributor
glouppe commented Nov 20, 2015

Fixed in #5858

@glouppe glouppe closed this as completed Nov 20, 2015
ogrisel added a commit that referenced this issue Jan 29, 2016
@ogrisel
DOC missing changelog entry for #5857
96d8e3c
ogrisel added a commit that referenced this issue Jan 29, 2016
@ogrisel
DOC missing changelog entry for #5857
2e270a7
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this issue Feb 13, 2016
@ogrisel
DOC missing changelog entry for scikit-learn#5857
5d943ed
yarikoptic added a commit to yarikoptic/scikit-learn that referenced this issue Feb 19, 2016
@yarikoptic
Merge tag '0.17.1' into releases
55dfe49 
* tag '0.17.1': (29 commits)
  Release 0.17.1
  MAINT remove non-existing cache folder in 0.17.X branch
  FIX cythonize TSNE
  MAINT simplify freeing logic for Barnes-Hut SNE memory leak fix
  Fix memory leak in Barnes-Hut SNE
  FIX check_build_doc.py false positive detections
  MAINT more informative output to circle/check_build_doc.py
  FIX fetch_california_housing
  FIX in randomized_svd flip sign
  Updated examples and tests that use scipy's lena
  DOC whats_new entry for scikit-learn#6258
  fix joblib error in LatentDirichletAllocation
  MAINT fix / speedup travis on 0.17.X
  MAINT Upgrade pip in appveyor and display version
  DOC missing changelog entry for scikit-learn#5857
  DOC add fix for scikit-learn#6147 to the changelog
  FIX 6147: ensure that AUC is always a float
  TST non-regression test for scikit-learn#6147, roc_auc on memmap data
  Added changelog entry about scikit-learn#6196
  Fix reading of bunch pickles
  ...
yarikoptic added a commit to yarikoptic/scikit-learn that referenced this issue Feb 19, 2016
@yarikoptic
Merge branch 'releases' into dfsg
1452704 
* releases: (29 commits)
  Release 0.17.1
  MAINT remove non-existing cache folder in 0.17.X branch
  FIX cythonize TSNE
  MAINT simplify freeing logic for Barnes-Hut SNE memory leak fix
  Fix memory leak in Barnes-Hut SNE
  FIX check_build_doc.py false positive detections
  MAINT more informative output to circle/check_build_doc.py
  FIX fetch_california_housing
  FIX in randomized_svd flip sign
  Updated examples and tests that use scipy's lena
  DOC whats_new entry for scikit-learn#6258
  fix joblib error in LatentDirichletAllocation
  MAINT fix / speedup travis on 0.17.X
  MAINT Upgrade pip in appveyor and display version
  DOC missing changelog entry for scikit-learn#5857
  DOC add fix for scikit-learn#6147 to the changelog
  FIX 6147: ensure that AUC is always a float
  TST non-regression test for scikit-learn#6147, roc_auc on memmap data
  Added changelog entry about scikit-learn#6196
  Fix reading of bunch pickles
  ...
yarikoptic added a commit to yarikoptic/scikit-learn that referenced this issue Feb 19, 2016
@yarikoptic
Merge branch 'dfsg' into debian
82c41fb 
* dfsg: (29 commits)
  Release 0.17.1
  MAINT remove non-existing cache folder in 0.17.X branch
  FIX cythonize TSNE
  MAINT simplify freeing logic for Barnes-Hut SNE memory leak fix
  Fix memory leak in Barnes-Hut SNE
  FIX check_build_doc.py false positive detections
  MAINT more informative output to circle/check_build_doc.py
  FIX fetch_california_housing
  FIX in randomized_svd flip sign
  Updated examples and tests that use scipy's lena
  DOC whats_new entry for scikit-learn#6258
  fix joblib error in LatentDirichletAllocation
  MAINT fix / speedup travis on 0.17.X
  MAINT Upgrade pip in appveyor and display version
  DOC missing changelog entry for scikit-learn#5857
  DOC add fix for scikit-learn#6147 to the changelog
  FIX 6147: ensure that AUC is always a float
  TST non-regression test for scikit-learn#6147, roc_auc on memmap data
  Added changelog entry about scikit-learn#6196
  Fix reading of bunch pickles
  ...
mannby pushed a commit to mannby/scikit-learn that referenced this issue Apr 22, 2016
@ogrisel
DOC missing changelog entry for scikit-learn#5857
e96b386
TomDLT pushed a commit to TomDLT/scikit-learn that referenced this issue Oct 3, 2016
@ogrisel @TomDLT
DOC missing changelog entry for scikit-learn#5857
74def36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@amueller @glouppe @jmschrei @mcculloh
0