Skip to content

[MRG] remove fit_params from BaseSearchCV.__init__#13519

Merged
adrinjalali merged 3 commits intoscikit-learn:masterfrom
NicolasHug:remove_fit_param_grid_search
Mar 27, 2019
Merged

[MRG] remove fit_params from BaseSearchCV.__init__#13519
adrinjalali merged 3 commits intoscikit-learn:masterfrom
NicolasHug:remove_fit_param_grid_search

Conversation

@NicolasHug
Copy link
Copy Markdown
Member

@NicolasHug NicolasHug commented Mar 25, 2019

fit_params in GridSearchCV and RandomizedSearchCV constructors was deprecated and is now removed for 0.21, but it wasn't removed from the base class BaseSearchCV

@NicolasHug
Copy link
Copy Markdown
Member Author

FYI, for some reason test_imputation.py was failing on windows so I added a few things in 72ff1b0 to make a test deterministic

Copy link
Copy Markdown
Member

@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @NicolasHug

@adrinjalali adrinjalali merged commit 7500693 into scikit-learn:master Mar 27, 2019
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
* remove fit_params from BaseSearchCV.__init__

* fixed randomness of test_imputation_pipeline_grid_search?

* removed KFolds
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
koenvandevelde pushed a commit to koenvandevelde/scikit-learn that referenced this pull request Jul 12, 2019
* remove fit_params from BaseSearchCV.__init__

* fixed randomness of test_imputation_pipeline_grid_search?

* removed KFolds
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.

3 participants