8000 Fixed parametrize variable naming for test_stratified_splitter_without_y · scikit-learn/scikit-learn@971240a · GitHub
[go: up one dir, main page]

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 971240a

Browse files
Anurag-Varmalesteve
andcommitted
Fixed parametrize variable naming for test_stratified_splitter_without_y
Co-authored-by: Loïc Estève <loic.esteve@ymail.com>
1 parent a02e4df commit 971240a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

sklearn/model_selection/tests/test_split.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686

8787
ALL_SPLITTERS = NO_GROUP_SPLITTERS + GROUP_SPLITTERS # type: ignore
8888

89-
splitters_requiring_y = [
89+
SPLITTERS_REQUIRING_TARGET = [
9090
StratifiedKFold(),
9191
StratifiedShuffleSplit(),
9292
RepeatedStratifiedKFold(),
@@ -2063,7 +2063,7 @@ def test_no_group_splitters_warns_with_groups(cv):
20632063

20642064

20652065
@pytest.mark.parametrize(
2066-
"cv", splitters_requiring_y, ids=[str(cv) for cv in splitters_requiring_y]
2066+
"cv", SPLITTERS_REQUIRING_TARGET, ids=[str(cv) for cv in SPLITTERS_REQUIRING_TARGET]
20672067
)
20682068
def test_stratified_splitter_without_y(cv):
20692069
msg = r"split.+missing 1 required positional argument: 'y'"

0 commit comments

Comments
 (0)
0