Fixed introduced test failures, and fixed unbounded recursion noted on PR.

Added few more tests pertaining to out keyword use

Author: oleksandr-pavlyk

dpctl/dptensor/numpy_usm_shared.py

@@ -83,10 +83,8 @@ def convert_ndarray_to_np_ndarray(x, require_ndarray=False):
         return np.array(x, copy=False, subok=False)
     elif isinstance(x, tuple):
         return tuple(
-            convert_ndarray_to_np_ndarray(
-                y,
-                require_ndarray=require_ndarray
-            ) for y in x)
+            convert_ndarray_to_np_ndarray(y, require_ndarray=require_ndarray) for y in x
+        )
     elif require_ndarray:
         raise TypeError
     else:
@@ -249,7 +247,7 @@ def __array_finalize__(self, obj):
 
     # Convert to a NumPy ndarray.
     def as_ndarray(self):
-        return np.copy(np.ndarray(self.shape, self.dtype, self))
+        return np.array(self, copy=True, subok=False)
 
     def __array__(self):
         return self
@@ -287,17 +285,22 @@ def __array_ufunc__(self, ufunc, method, *inputs, **kwargs):
                 # maybe copy?
                 # deal with multiple returned arrays, so kwargs['out'] can be tuple
                 res_type = np.result_type(*typing)
-                out = empty(inputs[0].shape, dtype=res_type)
-                out_as_np = np.ndarray(out.shape, out.dtype, out)
+                out_arg = empty(inputs[0].shape, dtype=res_type)
+                out_as_np = convert_ndarray_to_np_ndarray(out_arg)
                 kwargs["out"] = out_as_np
             else:
                 # If they manually gave numpy_usm_shared as out kwarg then we
                 # have to also cast as regular NumPy ndarray to avoid recursion.
                 try:
-                    kwargs["out"] = convert_ndarray_to_np_ndarray(out_arg, require_ndarray=True)
+                    kwargs["out"] = convert_ndarray_to_np_ndarray(
+                        out_arg, require_ndarray=True
+                    )
                 except TypeError:
-                    raise TypeError("Return arrays must each be {}".format(self.__class__))
-            return ufunc(*scalars, **kwargs)
+                    raise TypeError(
+                        "Return arrays must each be {}".format(self.__class__)
+                    )
+            ufunc(*scalars, **kwargs)
+            return out_arg
         elif method == "reduce":
             N = None
             scalars = []
@@ -333,7 +336,11 @@ def __array_function__(self, func, types, args, kwargs):
             cm = sys.modules[__name__]
             affunc = getattr(cm, fname)
             fargs = [x.view(np.ndarray) if isinstance(x, ndarray) else x for x in args]
-            return affunc(*fargs, **kwargs)
+            fkwargs = {
+                key: convert_ndarray_to_np_ndarray(val) for key, val in kwargs.items()
+            }
+            res = affunc(*fargs, **fkwargs)
+            return kwargs["out"] if "out" in kwargs else res
         return NotImplemented
 
 

dpctl/tests/test_dparray.py

@@ -61,6 +61,7 @@ def func_operation_with_const(dpctl_array):
     def test_dparray_mixing_dpctl_and_numpy(self):
         dp_numpy = numpy.ones((256, 4), dtype="d")
         res = dp_numpy * self.X
+        self.assertIsInstance(self.X, dparray.ndarray)
         self.assertIsInstance(res, dparray.ndarray)
 
     def test_dparray_shape(self):
@@ -79,6 +80,20 @@ def test_numpy_sum_with_dparray(self):
         res = numpy.sum(self.X)
         self.assertEqual(res, 1024.0)
 
+    def test_numpy_sum_with_dparray_out(self):
+        res = dparray.empty((self.X.shape[1],), dtype=self.X.dtype)
+        res2 = numpy.sum(self.X, axis=0, out=res)
+        self.assertTrue(res is res2)
+        self.assertIsInstance(res2, dparray.ndarray)
+
+    def test_frexp_with_out(self):
+        X = dparray.array([0.5, 4.7])
+        mant = dparray.empty((2,), dtype="d")
+        exp = dparray.empty((2,), dtype="i4")
+        res = numpy.frexp(X, out=(mant, exp))
+        self.assertTrue(res[0] is mant)
+        self.assertTrue(res[1] is exp)
+
 
 if __name__ == "__main__":
     unittest.main()