chore: precommit

Author: jharlow-intel

dpctl/_sycl_queue.pxd

@@ -104,7 +104,9 @@ cdef public api class SyclQueue (_SyclQueue) [
     cpdef memcpy(self, dest, src, size_t count)
     cpdef SyclEvent memcpy_async(self, dest, src, size_t count, list dEvents=*)
     cpdef copy(self, dest, src, size_t count, str dtype=*)
-    cpdef SyclEvent copy_async(self, dest, src, size_t count, list dEvents=*, str dtype=*)
+    cpdef SyclEvent copy_async(
+        self, dest, src, size_t count, list dEvents=*, str dtype=*
+    )
     cpdef prefetch(self, ptr, size_t count=*)
     cpdef mem_advise(self, ptr, size_t count, int mem)
     cpdef SyclEvent submit_barrier(self, dependent_events=*)

dpctl/_sycl_queue.pyx

@@ -467,7 +467,8 @@ ctypedef DPCTLSyclEventRef (*queue_copy_fn)(
 )
 
 ctypedef DPCTLSyclEventRef (*queue_copy_with_events_fn)(
-    const DPCTLSyclQueueRef, void*, const void*, size_t, const DPCTLSyclEventRef*, size_t
+    const DPCTLSyclQueueRef, void*, const void*, size_t,
+    const DPCTLSyclEventRef*, size_t
 )
 
 
@@ -511,7 +512,6 @@ cdef DPCTLSyclEventRef _copy_memcpy_impl(
     cdef bint dst_is_buf = False
     cdef int ret_code = 0
     cdef size_t element_size = 0
-    cdef size_t element_count = 0
 
     if dtype is not None:
         element_size = _get_dtype_size(dtype)
@@ -520,7 +520,6 @@ cdef DPCTLSyclEventRef _copy_memcpy_impl(
                 f"byte_count ({byte_count}) must be a multiple of dtype "
                 f"element size ({element_size} bytes for '{dtype}')"
             )
-        element_count = byte_count // element_size
 
     if isinstance(src, _Memory):
         c_src_ptr = <void*>(<_Memory>src).get_data_ptr()
@@ -1516,8 +1515,9 @@ cdef class SyclQueue(_SyclQueue):
             count (int):
                 Number of bytes to copy.
             dtype (str, optional):
-                Data type string (e.g., 'i4', 'f8') for typed copy validation.
-                If provided, validates that count is a multiple of the element size.
+                Data type string (e.g., 'i4', 'f8') for typed copy
+                validation. If provided, validates that count is a
+                multiple of the element size.
         """
         cdef DPCTLSyclEventRef ERef = NULL
 
@@ -1550,8 +1550,9 @@ cdef class SyclQueue(_SyclQueue):
             dEvents (List[dpctl.SyclEvent], optional):
                 Events that this copy depends on.
             dtype (str, optional):
-                Data type string (e.g., 'i4', 'f8') for typed copy validation.
-                If provided, validates that count is a multiple of the element size.
+                Data type string (e.g., 'i4', 'f8') for typed copy
+                validation. If provided, validates that count is a
+                multiple of the element size.
                 Supported types: i1, u1, i2, u2, i4, u4, i8, u8, f4, f8.
 
         Returns:

dpctl/tests/test_sycl_queue_copy.py

@@ -159,7 +159,7 @@ def test_copy_async_with_dtype_valid():
     ],
 )
 def test_copy_with_dtype_invalid_count(dtype, element_size, bad_count):
-    """Test that copy raises ValueError when count is not a multiple of dtype size."""
+    """Test copy raises ValueError when count isn't a dtype multiple."""
     try:
         q = dpctl.SyclQueue()
     except dpctl.SyclQueueCreationError:
@@ -209,9 +209,10 @@ def test_copy_with_invalid_dtype():
     for bad_dtype in invalid_dtypes:
         with pytest.raises(ValueError) as cm:
             q.copy(dst, src, nbytes, dtype=bad_dtype)
-        assert "dtype" in str(cm.value).lower() or "unrecognized" in str(
-            cm.value
-        ).lower()
+        assert (
+            "dtype" in str(cm.value).lower()
+            or "unrecognized" in str(cm.value).lower()
+        )
 
 
 def test_copy_with_dtype_host_buffers():