Explicitly raise an exception if the user tries to tune `n_estimators` with early stopping

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

Explicitly raise an exception if the user tries to tune `n_estimators` with early stopping #101

Closed richardliaw closed 3 years ago

richardliaw commented 4 years ago

Wouldn't it be a good idea to explicitly raise an exception if the user tries to tune n_estimators with early stopping? I worry that warnings will be unnoticed.

Originally posted by @Yard1 in https://github.com/ray-project/tune-sklearn/pull/90#issuecomment-691522918

This should perhaps be raised in base_search. We may also want to raise exceptions for other automatic parameter settings too.

cc @inventormc