Skip to content

feature_scoring.py: Change max_features in get_rf_feature_scores to "sqrt"#129

Merged
quaquel merged 1 commit intoquaquel:masterfrom
EwoutH:patch-9
May 18, 2022
Merged

feature_scoring.py: Change max_features in get_rf_feature_scores to "sqrt"#129
quaquel merged 1 commit intoquaquel:masterfrom
EwoutH:patch-9

Conversation

@EwoutH
Copy link
Copy Markdown
Collaborator

@EwoutH EwoutH commented May 18, 2022

The usage of max_features="auto" as parameter value in RandomForestRegressors and ExtraTreesRegressors is depreciated in scikit-learn 1.1 and will be removed in 1.3. The default value for max_features also changed from from "auto" to "sqrt" in version 1.1.

This commit changes max_features parameter to "sqrt". Behaviour should be identical. Note that this value can also be removed, since the default value is also "sqrt".

Closes #127.

…sqrt"

The usage of max_features="auto" as parameter value in RandomForestRegressors and ExtraTreesRegressors is depreciated in scikit-learn 1.1 and will be removed in 1.3. The default value for max_features also changed from from "auto" to "sqrt" in version 1.1.

This commit changes max_features parameter to "sqrt". Behaviour should be identical. Note that this value can also be removed, since the default value is also "sqrt".
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 80.466% when pulling 6cb9131 on EwoutH:patch-9 into c1da7cf on quaquel:master.

@quaquel quaquel merged commit 5e03fd9 into quaquel:master May 18, 2022
@EwoutH EwoutH mentioned this pull request May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

max_features="auto" depreciated in RandomForestClassifier used in feature_scoring.py

3 participants