Numba sparse: Remove codebase xfails

Author: ricardoV94

tests/compile/function/test_pfunc.py

@@ -3,14 +3,13 @@
 import scipy as sp
 
 import pytensor.tensor as pt
-from pytensor.compile import UnusedInputError, get_default_mode, get_mode
+from pytensor.compile import UnusedInputError, get_mode
 from pytensor.compile.function import function, pfunc
 from pytensor.compile.function.pfunc import rebuild_collect_shared
 from pytensor.compile.io import In
 from pytensor.compile.sharedvalue import shared
 from pytensor.configdefaults import config
 from pytensor.graph.utils import MissingInputError
-from pytensor.link.numba import NumbaLinker
 from pytensor.sparse import SparseTensorType
 from pytensor.tensor.math import sum as pt_sum
 from pytensor.tensor.type import (
@@ -766,10 +765,6 @@ def test_shared_constructor_copies(self):
         # rule #2 reading back from pytensor-managed memory
         assert not np.may_share_memory(A.get_value(borrow=False), data_of(A))
 
-    @pytest.mark.xfail(
-        condition=isinstance(get_default_mode().linker, NumbaLinker),
-        reason="Numba does not support Sparse Ops yet",
-    )
     def test_sparse_input_aliasing_affecting_inplace_operations(self):
         # Note: to trigger this bug with pytensor rev 4586:2bc6fc7f218b,
         #        you need to make in inputs mutable (so that inplace

tests/sparse/__init__.py

@@ -1,11 +0,0 @@
-import pytest
-
-from pytensor.compile import get_default_mode
-from pytensor.link.numba import NumbaLinker
-
-
-if isinstance(get_default_mode().linker, NumbaLinker):
-    pytest.skip(
-        reason="Numba does not support Sparse Ops yet",
-        allow_module_level=True,
-    )

tests/sparse/test_math.py

@@ -8,7 +8,9 @@
 import pytensor
 import pytensor.sparse.math as psm
 import pytensor.tensor as pt
+from pytensor.compile import get_default_mode
 from pytensor.configdefaults import config
+from pytensor.link.numba import NumbaLinker
 from pytensor.scalar import upcast
 from pytensor.sparse.basic import (
     CSR,
@@ -427,33 +429,54 @@ def test_opt_unpack(self):
         )
         f(kernvals, imvals)
 
-    def test_dot_sparse_sparse(self):
+    @pytest.mark.parametrize(
+        "sparse_format_a",
+        (
+            "csc",
+            "csr",
+            pytest.param(
+                "bsr",
+                marks=pytest.mark.xfail(
+                    isinstance(get_default_mode().linker, NumbaLinker),
+                    reason="Numba does not support bsr",
+                ),
+            ),
+        ),
+    )
+    @pytest.mark.parametrize(
+        "sparse_format_b",
+        (
+            "csc",
+            "csr",
+            pytest.param(
+                "bsr",
+                marks=pytest.mark.xfail(
+                    isinstance(get_default_mode().linker, NumbaLinker),
+                    reason="Numba does not support bsr",
+                ),
+            ),
+        ),
+    )
+    def test_dot_sparse_sparse(self, sparse_format_a, sparse_format_b):
         sparse_dtype = "float64"
         sp_mat = {
             "csc": scipy_sparse.csc_matrix,
             "csr": scipy_sparse.csr_matrix,
             "bsr": scipy_sparse.csr_matrix,
         }
-
-        for sparse_format_a in ["csc", "csr", "bsr"]:
-            for sparse_format_b in ["csc", "csr", "bsr"]:
-                a = SparseTensorType(sparse_format_a, dtype=sparse_dtype)()
-                b = SparseTensorType(sparse_format_b, dtype=sparse_dtype)()
-                d = pt.dot(a, b)
-                f = pytensor.function([a, b], d)
-                for M, N, K, nnz in [
-                    (4, 3, 2, 3),
-                    (40, 30, 20, 3),
-                    (40, 30, 20, 30),
-                    (400, 3000, 200, 6000),
-                ]:
-                    a_val = sp_mat[sparse_format_a](
-                        random_lil((M, N), sparse_dtype, nnz)
-                    )
-                    b_val = sp_mat[sparse_format_b](
-                        random_lil((N, K), sparse_dtype, nnz)
-                    )
-                    f(a_val, b_val)
+        a = SparseTensorType(sparse_format_a, dtype=sparse_dtype)()
+        b = SparseTensorType(sparse_format_b, dtype=sparse_dtype)()
+        d = pt.dot(a, b)
+        f = pytensor.function([a, b], d)
+        for M, N, K, nnz in [
+            (4, 3, 2, 3),
+            (40, 30, 20, 3),
+            (40, 30, 20, 30),
+            (400, 3000, 200, 6000),
+        ]:
+            a_val = sp_mat[sparse_format_a](random_lil((M, N), sparse_dtype, nnz))
+            b_val = sp_mat[sparse_format_b](random_lil((N, K), sparse_dtype, nnz))
+            f(a_val, b_val)  # TODO: Test something
 
     def test_tensor_dot_types(self):
         x = csc_matrix("x")

tests/sparse/test_rewriting.py

@@ -7,6 +7,7 @@
 from pytensor import sparse
 from pytensor.compile.mode import Mode, get_default_mode
 from pytensor.configdefaults import config
+from pytensor.link.numba import NumbaLinker
 from pytensor.sparse.rewriting import SamplingDotCSR, sd_csc
 from pytensor.tensor.basic import as_tensor_variable
 from pytensor.tensor.math import sum as pt_sum
@@ -68,6 +69,10 @@ def test_local_csm_grad_c():
 @pytest.mark.skipif(
     not pytensor.config.cxx, reason="G++ not available, so we need to skip this test."
 )
+@pytest.mark.skipif(
+    isinstance(get_default_mode().linker, NumbaLinker),
+    reason="This is a C-specific test",
+)
 def test_local_mul_s_d():
     mode = get_default_mode()
     mode = mode.including("specialize", "local_mul_s_d")
@@ -86,6 +91,10 @@ def test_local_mul_s_d():
 @pytest.mark.skipif(
     not pytensor.config.cxx, reason="G++ not available, so we need to skip this test."
 )
+@pytest.mark.skipif(
+    isinstance(get_default_mode().linker, NumbaLinker),
+    reason="This is a C-specific test",
+)
 def test_local_mul_s_v():
     mode = get_default_mode()
     mode = mode.including("specialize", "local_mul_s_v")
@@ -104,6 +113,10 @@ def test_local_mul_s_v():
 @pytest.mark.skipif(
     not pytensor.config.cxx, reason="G++ not available, so we need to skip this test."
 )
+@pytest.mark.skipif(
+    isinstance(get_default_mode().linker, NumbaLinker),
+    reason="This is a C-specific test",
+)
 def test_local_structured_add_s_v():
     mode = get_default_mode()
     mode = mode.including("specialize", "local_structured_add_s_v")
@@ -122,6 +135,10 @@ def test_local_structured_add_s_v():
 @pytest.mark.skipif(
     not pytensor.config.cxx, reason="G++ not available, so we need to skip this test."
 )
+@pytest.mark.skipif(
+    isinstance(get_default_mode().linker, NumbaLinker),
+    reason="This is a C-specific test",
+)
 def test_local_sampling_dot_csr():
     mode = get_default_mode()
     mode = mode.including("specialize", "local_sampling_dot_csr")

tests/test_raise_op.py

@@ -4,10 +4,9 @@
 
 import pytensor
 import pytensor.tensor as pt
-from pytensor.compile.mode import OPT_FAST_RUN, Mode, get_default_mode
+from pytensor.compile.mode import OPT_FAST_RUN, Mode
 from pytensor.graph import vectorize_graph
 from pytensor.graph.basic import Constant, equal_computations
-from pytensor.link.numba import NumbaLinker
 from pytensor.raise_op import Assert, CheckAndRaise, assert_op
 from pytensor.scalar.basic import ScalarType, float64
 from pytensor.sparse import as_sparse_variable
@@ -182,10 +181,6 @@ def test_infer_shape_scalar(self):
         )
 
 
-@pytest.mark.xfail(
-    condition=isinstance(get_default_mode().linker, NumbaLinker),
-    reason="Numba does not support Sparse Ops yet",
-)
 def test_CheckAndRaise_sparse_variable():
     check_and_raise = CheckAndRaise(ValueError, "sparse_check")
 

tests/typed_list/test_basic.py

@@ -7,8 +7,6 @@
 import pytensor
 import pytensor.typed_list
 from pytensor import sparse
-from pytensor.compile import get_default_mode
-from pytensor.link.numba import NumbaLinker
 from pytensor.tensor.type import (
     TensorType,
     integer_dtypes,
@@ -454,10 +452,6 @@ def test_non_tensor_type(self):
 
         assert f([[x, y], [x, y, y]], [x, y]) == 0
 
-    @pytest.mark.xfail(
-        condition=isinstance(get_default_mode().linker, NumbaLinker),
-        reason="Numba does not support Sparse Ops yet",
-    )
     def test_sparse(self):
         mySymbolicSparseList = TypedListType(
             sparse.SparseTensorType("csr", pytensor.config.floatX)
@@ -525,10 +519,6 @@ def test_non_tensor_type(self):
 
         assert f([[x, y], [x, y, y]], [x, y]) == 1
 
-    @pytest.mark.xfail(
-        condition=isinstance(get_default_mode().linker, NumbaLinker),
-        reason="Numba does not support Sparse Ops yet",
-    )
     def test_sparse(self):
         mySymbolicSparseList = TypedListType(
             sparse.SparseTensorType("csr", pytensor.config.floatX)