-
-
Notifications
You must be signed in to change notification settings - Fork 25.2k
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
API Rename base_estimator
into estimator
in ClassifierChain
, and RegressorChain
#29682
base: main
Are you sure you want to change the base?
API Rename base_estimator
into estimator
in ClassifierChain
, and RegressorChain
#29682
Conversation
❌ Linting issuesThis PR is introducing linting issues. Here's a summary of the issues. Note that you can avoid having linting issues by enabling You can see the details of the linting issues under the
|
Closes #29620
This PR renames
base_estimator
intoestimator
inClassifierChain
, andRegressorChain
such that they are consistent with other classifiers and regressors in the repository