MAINT Extend message for large sparse matrices support (#25961)

jjerphan · Micky774 · web-flow · commit 545cc809e160 · 2023-03-25T19:25:47.000Z
Co-authored-by: Meekail Zain &lt;34613774+Micky774@users.noreply.github.com&gt;
diff --git a/sklearn/utils/tests/test_validation.py b/sklearn/utils/tests/test_validation.py
@@ -632,7 +632,7 @@ def test_check_array_accept_large_sparse_raise_exception(X_64bit):
     # When large sparse are not allowed
     msg = (
         "Only sparse matrices with 32-bit integer indices "
-        "are accepted. Got int64 indices."
+        "are accepted. Got int64 indices. Please do report"
     )
     with pytest.raises(ValueError, match=msg):
         check_array(X_64bit, accept_sparse=True, accept_large_sparse=False)
diff --git a/sklearn/utils/validation.py b/sklearn/utils/validation.py
@@ -987,8 +987,11 @@ def _check_large_sparse(X, accept_large_sparse=False):
             indices_datatype = getattr(X, key).dtype
             if indices_datatype not in supported_indices:
                 raise ValueError(
-                    "Only sparse matrices with 32-bit integer"
-                    " indices are accepted. Got %s indices." % indices_datatype
+                    "Only sparse matrices with 32-bit integer indices are accepted."
+                    f" Got {indices_datatype} indices. Please do report a minimal"
+                    " reproducer on scikit-learn issue tracker so that support for"
+                    " your use-case can be studied by maintainers. See:"
+                    " https://scikit-learn.org/dev/developers/minimal_reproducer.html"
                 )
 
 

Original file line number	Diff line number	Diff line change
`@@ -632,7 +632,7 @@ def test_check_array_accept_large_sparse_raise_exception(X_64bit):`
`632`	`632`	`# When large sparse are not allowed`
`633`	`633`	`msg = (`
`634`	`634`	`"Only sparse matrices with 32-bit integer indices "`
`635`		`- "are accepted. Got int64 indices."`
	`635`	`+ "are accepted. Got int64 indices. Please do report"`
`636`	`636`	`)`
`637`	`637`	`with pytest.raises(ValueError, match=msg):`
`638`	`638`	`check_array(X_64bit, accept_sparse=True, accept_large_sparse=False)`