Remove support for non-scalar period

Author: vlad-perevezentsev

dpnp/dpnp_iface_mathematical.py

@@ -2813,7 +2813,7 @@ def interp(x, xp, fp, left=None, right=None, period=None):
 
         Default: ``fp[-1]``.
 
-    period : {None, scalar, dpnp.ndarray, usm_ndarray}, optional
+    period : {None, scalar}, optional
         A period for the x-coordinates. This parameter allows the proper
         interpolation of angular x-coordinates. Parameters `left` and `right`
         are ignored if `period` is specified.
@@ -2902,9 +2902,12 @@ def interp(x, xp, fp, left=None, right=None, period=None):
     fp = dpnp.asarray(fp, dtype=out_dtype, order="C")
 
     if period is not None:
-        period = _validate_interp_param(period, "period", exec_q, usm_type)
+        if not dpnp.isscalar(period):
+            raise TypeError(f"period must be a scalar, but got {type(period)}")
         if period == 0:
             raise ValueError("period must be a non-zero value")
+        period = _validate_interp_param(period, "period", exec_q, usm_type)
+
         period = dpnp.abs(period)
 
         # left/right are ignored when period is specified

dpnp/tests/test_mathematical.py

@@ -1273,16 +1273,6 @@ def test_errors(self):
         # period is not scalar or 0-dim
         assert_raises(TypeError, dpnp.interp, x, xp, fp, period=[180])
 
-        # period has a different SYCL queue
-        q1 = dpctl.SyclQueue()
-        q2 = dpctl.SyclQueue()
-
-        x = dpnp.array([1.0], sycl_queue=q1)
-        xp = dpnp.array([0.0, 2.0], sycl_queue=q1)
-        fp = dpnp.array([0.0, 1.0], sycl_queue=q1)
-        period = dpnp.array([180], sycl_queue=q2)
-        assert_raises(ValueError, dpnp.interp, x, xp, fp, period=period)
-
         # left is not scalar or 0-dim
         left = dpnp.array([1.0])
         assert_raises(ValueError, dpnp.interp, x, xp, fp, left=left)
@@ -1292,6 +1282,8 @@ def test_errors(self):
         assert_raises(ValueError, dpnp.interp, x, xp, fp, left=left)
 
         # left has a different SYCL queue
+        q1 = dpctl.SyclQueue()
+        q2 = dpctl.SyclQueue()
         left = dpnp.array(1.0, sycl_queue=q2)
         if q1 != q2:
             assert_raises(ValueError, dpnp.interp, x, xp, fp, left=left)

dpnp/tests/test_sycl_queue.py

@@ -1464,9 +1464,7 @@ def test_interp(device, left, right, period):
 
     l = None if left is None else dpnp.array(left, sycl_queue=x.sycl_queue)
     r = None if right is None else dpnp.array(right, sycl_queue=x.sycl_queue)
-    p = None if period is None else dpnp.array(period, sycl_queue=x.sycl_queue)
-
-    result = dpnp.interp(x, xp, fp, left=l, right=r, period=p)
+    result = dpnp.interp(x, xp, fp, left=l, right=r, period=period)
 
     assert_sycl_queue_equal(result.sycl_queue, x.sycl_queue)
 

dpnp/tests/test_usm_type.py

@@ -1311,21 +1311,6 @@ def test_left_right(self, usm_type_x, usm_type_left, usm_type_right):
             ]
         )
 
-    @pytest.mark.parametrize("usm_type_x", list_of_usm_types)
-    @pytest.mark.parametrize("usm_type_period", list_of_usm_types)
-    def test_period(self, usm_type_x, usm_type_period):
-        x = dpnp.linspace(0.1, 9.9, 20, usm_type=usm_type_x)
-        xp = dpnp.linspace(0.0, 10.0, 5, usm_type=usm_type_x)
-        fp = dpnp.array(xp * 2 + 1, usm_type=usm_type_x)
-        period = dpnp.array(10.0, usm_type=usm_type_period)
-
-        result = dpnp.interp(x, xp, fp, period=period)
-
-        assert period.usm_type == usm_type_period
-        assert result.usm_type == du.get_coerced_usm_type(
-            [x.usm_type, xp.usm_type, fp.usm_type, period.usm_type]
-        )
-
 
 @pytest.mark.parametrize("usm_type", list_of_usm_types)
 class TestLinAlgebra: