ray-project / tune-sklearn

A drop-in replacement for Scikit-Learn’s GridSearchCV / RandomizedSearchCV -- but with cutting edge hyperparameter tuning techniques.
https://docs.ray.io/en/master/tune/api_docs/sklearn.html
Apache License 2.0
465 stars 52 forks source link

Avoid Loky Pickle Error #150

Closed richardliaw closed 3 years ago

richardliaw commented 3 years ago

It seems like in newer versions of sklearn, we are unable to use joblib inside Ray.

This change makes it so that users will avoid errors in the future.

Closes https://github.com/ray-project/tune-sklearn/issues/135

Yard1 commented 3 years ago

I believe it would be a good idea to add a warning if sk_n_jobs has been set by the user (in old code), to inform them that it is ignored due to issues with sklearn and defaulted to 1.

richardliaw commented 3 years ago

ah, great point!

richardliaw commented 3 years ago

@Yard1 raised a deprecation message instead!