8000 Clarify error message for min_samples_split. (#8167) · maskani-moh/scikit-learn@c4fc08c · GitHub
[go: up one dir, main page]

Skip to content

Commit c4fc08c

Browse files
mikebenfieldmaskani-moh
authored andcommitted
Clarify error message for min_samples_split. (scikit-learn#8167)
1 parent e395894 commit c4fc08c

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

sklearn/tree/tree.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -185,14 +185,16 @@ def fit(self, X, y, sample_weight=None, check_input=True,
185185

186186
if isinstance(self.min_samples_split, (numbers.Integral, np.integer)):
187187
if not 2 <= self.min_samples_split:
188-
raise ValueError("min_samples_split must be at least 2 "
189-
"or in (0, 1.0], got %s"
188+
raise ValueError("min_samples_split must be an integer "
189+
"greater than 1 or a float in (0.0, 1.0]; "
190+
"got the integer %s"
190191
% self.min_samples_split)
191192
min_samples_split = self.min_samples_split
192193
else: # float
193194
if not 0. < self.min_samples_split <= 1.:
194-
raise ValueError("min_samples_split must be at least 2 "
195-
"or in (0, 1.0], got %s"
195+
raise ValueError("min_samples_split must be an integer "
196+
"greater than 1 or a float in (0.0, 1.0]; "
197+
"got the float %s"
196198
% self.min_samples_split)
197199
min_samples_split = int(ceil(self.min_samples_split * n_samples))
198200
min_samples_split = max(2, min_samples_split)

0 commit comments

Comments
 (0)
0