Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

InvalidParameterError when using ExtraTreesRegressor #1137

Open
chrisk314 opened this issue Dec 12, 2022 · 1 comment · May be fixed by #1184
Open

InvalidParameterError when using ExtraTreesRegressor #1137

chrisk314 opened this issue Dec 12, 2022 · 1 comment · May be fixed by #1184

Comments

@chrisk314
Copy link

As per the title, when attempting to use the ExtraTreesRegressor class an sklearn.utils._param_validation.InvalidParameterError exception is raised as below.

sklearn.utils._param_validation.InvalidParameterError: The 'criterion' parameter of ExtraTreesRegressor must be a str among {'squared_error', 'poisson', 'friedman_mse', 'absolute_error'}. Got 'mse' instead.

The ExtraTreesRegressor.criterion attribute is set using a default optional argument to the constructor with the value mse. It looks like this criterion value was removed in sklearn 1.2 as per this commit.

It should be an easy fix. Just requires changing the default argument for criterion to either friedman_mse or squared_error I guess... Not sure which one of these would be most appropriate as the default...
@froggleston
Copy link

According to the sklearn docs, mse should be replaced with squared_error.

@froggleston froggleston mentioned this issue Dec 13, 2022
Closed
@xmatthias xmatthias mentioned this issue Dec 13, 2022
Merged
@xmatthias xmatthias mentioned this issue Jan 30, 2023
Closed
@eccabay eccabay mentioned this issue Feb 8, 2023
Merged
@xmatthias xmatthias mentioned this issue Mar 13, 2023
Closed
@xmatthias xmatthias mentioned this issue Jul 5, 2023
Closed
@xmatthias xmatthias mentioned this issue Sep 25, 2023
Closed
ddelange added a commit to ddelange/scikit-optimize that referenced this issue Oct 23, 2023
@ddelange
Replace occurrences of mse with squared_error
809d5d3 
ref scikit-optimize#1137
@ddelange ddelange linked a pull request Oct 23, 2023 that will close this issue
Open
This was referenced Nov 6, 2023
Merged
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace occurrences of mse with squared_error ddelange/scikit-optimize
2 participants
@froggleston @chrisk314