Skip to content
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

Fix decision tree parameters #88

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

xtannier
Copy link
Contributor

Fixed a bug that appears from version 0.18 of scikit learn:
DecisionTreeClassifier parameters min_samples_split and min_samples_leaf expect either float values between 0 and 1 or integer values.
Float values higher than 1 were passed.

Xavier Tannier and others added 4 commits October 30, 2017 15:48
…sive classifiers.

- Removed n_iter parameter
- Added space search for parameters max_iter and tol.
- Fixed networkx dependency, following hyperopt#85
As they say, this should be considered a temporary fix until hyperopt creates a new pip release.
DecisionTreeClassifier parameters min_samples_split and min_samples_leaf expect either float values between 0 and 1 or integer values.
Float values higher than 1 were passed.
DecisionTreeClassifier parameters min_samples_split and min_samples_leaf expect either float values between 0 and 1 or integer values.
Float values higher than 1 were passed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant