Skip to content

Allow **fit_params on RFECV #17954

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
ciberger opened this issue Jul 20, 2020 · 5 comments
Closed

Allow **fit_params on RFECV #17954

ciberger opened this issue Jul 20, 2020 · 5 comments
Labels

Comments

@ciberger
Copy link

Hi,

minor change in fit method enabling extra fit parameters (i.e. **fit_params).

Link to source code:
https://github.com/scikit-learn/scikit-learn/blob/fd237278e/sklearn/feature_selection/_rfe.py#L486

Thanks!

@thomasjpfan
Copy link
Member

This would first require RFE to support **fit_params.

There is related work for getting RFE to support sample weights: #7333

@cerlymarco
Copy link

Everyone looking for RFE with Gradient Boosting, like LGBM or XGB, I suggest shap-hypetune... A python package for simultaneous Hyperparameters Tuning and Features Selection for Gradient Boosting Models.

It supports RFE (also with shap feature ranking) with every fitting parameters like in the standard algorithm API

@fingoldo
Copy link

For RFE, **fit_params are implemented & merged with #20380. Can someone enable them in RFECV as well, please?

@thomasjpfan
Copy link
Member

I'm marking this has Hard based on #24004 (comment) where we want to use the metadata routing API for routing the fit_params.

@adrinjalali
Copy link
Member

solved now with metadata routing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants