Various fixes done by isort and new black config.

Author: Diptorup Deb

docs/conf.in

@@ -41,7 +41,8 @@ use_doxyrest = "@DPCTL_ENABLE_DOXYREST@"
 
 if use_doxyrest == "ON":
     # Specify the path to Doxyrest extensions for Sphinx:
-    import sys, os
+    import os
+    import sys
 
     sys.path.insert(
         1,
@@ -168,9 +169,8 @@ class ClassMembersDocumenter(ClassDocumenter):
 # members and attributes.
 # See https://stackoverflow.com/questions/20569011/python-sphinx-autosummary-automated-listing-of-member-functions
 
-from sphinx.ext.autosummary import Autosummary
-from sphinx.ext.autosummary import get_documenter
 from docutils.parsers.rst import directives
+from sphinx.ext.autosummary import Autosummary, get_documenter
 from sphinx.util.inspect import safe_getattr
 
 

dpctl/__init__.py

@@ -24,8 +24,6 @@
 """
 __author__ = "Intel Corp."
 
-from .enum_types import *
-from .enum_types import __all__ as _enum_types_all__
 from dpctl._sycl_context import *
 from dpctl._sycl_context import __all__ as _sycl_context__all__
 from dpctl._sycl_device import *
@@ -40,8 +38,10 @@
 from dpctl._sycl_queue import __all__ as _sycl_queue__all__
 from dpctl._sycl_queue_manager import *
 from dpctl._sycl_queue_manager import __all__ as _sycl_qm__all__
-from ._version import get_versions
 
+from ._version import get_versions
+from .enum_types import *
+from .enum_types import __all__ as _enum_types_all__
 
 __all__ = (
     _sycl_context__all__

dpctl/_backend.pxd

@@ -21,8 +21,8 @@
 types defined by dpctl's C API.
 """
 
+from libc.stdint cimport int64_t, uint32_t
 from libcpp cimport bool
-from libc.stdint cimport uint32_t, int64_t
 
 
 cdef extern from "dpctl_error_handler_type.h":

dpctl/_sycl_context.pxd

@@ -20,9 +20,11 @@
 """ This file declares the SyclContext extension type.
 """
 
+from libcpp cimport bool
+
 from ._backend cimport DPCTLSyclContextRef
 from ._sycl_device cimport SyclDevice
-from libcpp cimport bool
+
 
 cdef class _SyclContext:
     """ Data owner for SyclContext

dpctl/_sycl_context.pyx

@@ -21,31 +21,34 @@
 """
 
 from __future__ import print_function
+
 import logging
-from ._backend cimport(
-    DPCTLSyclContextRef,
-    DPCTLSyclDeviceRef,
+
+from cpython cimport pycapsule
+from cpython.mem cimport PyMem_Free, PyMem_Malloc
+
+from ._backend cimport (
+    DPCTLContext_AreEq,
+    DPCTLContext_Copy,
     DPCTLContext_Create,
     DPCTLContext_CreateFromDevices,
+    DPCTLContext_Delete,
     DPCTLContext_DeviceCount,
     DPCTLContext_GetDevices,
-    DPCTLContext_Copy,
-    DPCTLContext_Delete,
-    DPCTLContext_AreEq,
-    DPCTLDevice_Delete,
     DPCTLDevice_Copy,
-    DPCTLDeviceVectorRef,
+    DPCTLDevice_Delete,
+    DPCTLDeviceMgr_GetCachedContext,
     DPCTLDeviceVector_CreateFromArray,
+    DPCTLDeviceVector_Delete,
     DPCTLDeviceVector_GetAt,
     DPCTLDeviceVector_Size,
-    DPCTLDeviceVector_Delete,
+    DPCTLDeviceVectorRef,
+    DPCTLSyclContextRef,
+    DPCTLSyclDeviceRef,
     error_handler_callback,
-    DPCTLDeviceMgr_GetCachedContext,
 )
-from ._sycl_queue cimport default_async_error_handler
 from ._sycl_device cimport SyclDevice
-from cpython.mem cimport PyMem_Malloc, PyMem_Free
-from cpython cimport pycapsule
+from ._sycl_queue cimport default_async_error_handler
 
 __all__ = [
     "SyclContext",

dpctl/_sycl_device.pxd

@@ -20,12 +20,13 @@
 """ This file declares the SyclDevice extension type.
 """
 
+from libcpp cimport bool as cpp_bool
+
 from ._backend cimport (
     DPCTLSyclDeviceRef,
     DPCTLSyclDeviceSelectorRef,
-    _partition_affinity_domain_type
+    _partition_affinity_domain_type,
 )
-from libcpp cimport bool as cpp_bool
 
 
 cdef class _SyclDevice:

dpctl/_sycl_device.pyx

@@ -21,70 +21,73 @@
 """
 
 from ._backend cimport (
-    _aspect_type,
-    _backend_type,
-    _device_type,
-    _partition_affinity_domain_type,
     DPCTLCString_Delete,
     DPCTLDefaultSelector_Create,
+    DPCTLDevice_AreEq,
     DPCTLDevice_Copy,
     DPCTLDevice_CreateFromSelector,
+    DPCTLDevice_CreateSubDevicesByAffinity,
+    DPCTLDevice_CreateSubDevicesByCounts,
+    DPCTLDevice_CreateSubDevicesEqually,
     DPCTLDevice_Delete,
-    DPCTLDeviceVectorRef,
-    DPCTLDeviceVector_Delete,
-    DPCTLDeviceVector_GetAt,
-    DPCTLDeviceVector_Size,
     DPCTLDevice_GetBackend,
-    DPCTLDevice_AreEq,
     DPCTLDevice_GetDeviceType,
     DPCTLDevice_GetDriverVersion,
+    DPCTLDevice_GetImage2dMaxHeight,
+    DPCTLDevice_GetImage2dMaxWidth,
+    DPCTLDevice_GetImage3dMaxDepth,
+    DPCTLDevice_GetImage3dMaxHeight,
+    DPCTLDevice_GetImage3dMaxWidth,
     DPCTLDevice_GetMaxComputeUnits,
     DPCTLDevice_GetMaxNumSubGroups,
+    DPCTLDevice_GetMaxReadImageArgs,
     DPCTLDevice_GetMaxWorkGroupSize,
     DPCTLDevice_GetMaxWorkItemDims,
     DPCTLDevice_GetMaxWorkItemSizes,
-    DPCTLDevice_GetVendor,
+    DPCTLDevice_GetMaxWriteImageArgs,
     DPCTLDevice_GetName,
+    DPCTLDevice_GetParentDevice,
+    DPCTLDevice_GetPreferredVectorWidthChar,
+    DPCTLDevice_GetPreferredVectorWidthDouble,
+    DPCTLDevice_GetPreferredVectorWidthFloat,
+    DPCTLDevice_GetPreferredVectorWidthHalf,
+    DPCTLDevice_GetPreferredVectorWidthInt,
+    DPCTLDevice_GetPreferredVectorWidthLong,
+    DPCTLDevice_GetPreferredVectorWidthShort,
+    DPCTLDevice_GetSubGroupIndependentForwardProgress,
+    DPCTLDevice_GetVendor,
+    DPCTLDevice_HasAspect,
     DPCTLDevice_IsAccelerator,
     DPCTLDevice_IsCPU,
     DPCTLDevice_IsGPU,
     DPCTLDevice_IsHost,
-    DPCTLDeviceMgr_PrintDeviceInfo,
     DPCTLDeviceMgr_GetRelativeId,
-    DPCTLFilterSelector_Create,
+    DPCTLDeviceMgr_PrintDeviceInfo,
     DPCTLDeviceSelector_Delete,
     DPCTLDeviceSelector_Score,
+    DPCTLDeviceVector_Delete,
+    DPCTLDeviceVector_GetAt,
+    DPCTLDeviceVector_Size,
+    DPCTLDeviceVectorRef,
+    DPCTLFilterSelector_Create,
     DPCTLSize_t_Array_Delete,
     DPCTLSyclBackendType,
     DPCTLSyclDeviceRef,
     DPCTLSyclDeviceSelectorRef,
-    DPCTLDevice_HasAspect,
     DPCTLSyclDeviceType,
-    DPCTLDevice_GetMaxReadImageArgs,
-    DPCTLDevice_GetMaxWriteImageArgs,
-    DPCTLDevice_GetImage2dMaxWidth,
-    DPCTLDevice_GetImage2dMaxHeight,
-    DPCTLDevice_GetImage3dMaxWidth,
-    DPCTLDevice_GetImage3dMaxHeight,
-    DPCTLDevice_GetImage3dMaxDepth,
-    DPCTLDevice_GetSubGroupIndependentForwardProgress,
-    DPCTLDevice_GetPreferredVectorWidthChar,
-    DPCTLDevice_GetPreferredVectorWidthShort,
-    DPCTLDevice_GetPreferredVectorWidthInt,
-    DPCTLDevice_GetPreferredVectorWidthLong,
-    DPCTLDevice_GetPreferredVectorWidthFloat,
-    DPCTLDevice_GetPreferredVectorWidthDouble,
-    DPCTLDevice_GetPreferredVectorWidthHalf,
-    DPCTLDevice_CreateSubDevicesEqually,
-    DPCTLDevice_CreateSubDevicesByCounts,
-    DPCTLDevice_CreateSubDevicesByAffinity,
-    DPCTLDevice_GetParentDevice,
+    _aspect_type,
+    _backend_type,
+    _device_type,
+    _partition_affinity_domain_type,
 )
+
 from . import backend_type, device_type
-from libc.stdint cimport uint32_t, int64_t
-from libc.stdlib cimport malloc, free
-import warnings
+
+from libc.stdint cimport int64_t, uint32_t
+from libc.stdlib cimport free, malloc
+
 import collections
+import warnings
 
 __all__ = [
     "SyclDevice",

dpctl/_sycl_device_factory.pxd

@@ -23,8 +23,10 @@ specific backend or device_type.
 """
 
 from libcpp cimport bool as cpp_bool
+
 from ._sycl_device cimport SyclDevice
 
+
 cpdef SyclDevice select_accelerator_device()
 cpdef SyclDevice select_cpu_device()
 cpdef SyclDevice select_default_device()

dpctl/_sycl_device_factory.pyx

@@ -26,27 +26,29 @@
 """
 
 from ._backend cimport (
-    _backend_type,
-    _device_type,
     DPCTLAcceleratorSelector_Create,
     DPCTLCPUSelector_Create,
     DPCTLDefaultSelector_Create,
     DPCTLDevice_CreateFromSelector,
     DPCTLDeviceMgr_GetDevices,
     DPCTLDeviceMgr_GetNumDevices,
     DPCTLDeviceSelector_Delete,
-    DPCTLDeviceVectorRef,
     DPCTLDeviceVector_Delete,
     DPCTLDeviceVector_GetAt,
     DPCTLDeviceVector_Size,
+    DPCTLDeviceVectorRef,
     DPCTLGPUSelector_Create,
     DPCTLHostSelector_Create,
     DPCTLSyclBackendType,
     DPCTLSyclDeviceRef,
     DPCTLSyclDeviceSelectorRef,
     DPCTLSyclDeviceType,
+    _backend_type,
+    _device_type,
 )
-from . import backend_type, device_type as device_type_t
+
+from . import backend_type
+from . import device_type as device_type_t
 
 __all__ = [
     "get_devices",

dpctl/_sycl_event.pyx

@@ -21,8 +21,10 @@
 """
 
 from __future__ import print_function
+
 import logging
-from ._backend cimport DPCTLSyclEventRef, DPCTLEvent_Delete, DPCTLEvent_Wait
+
+from ._backend cimport DPCTLEvent_Delete, DPCTLEvent_Wait, DPCTLSyclEventRef
 
 __all__ = [
     "SyclEvent",

dpctl/_sycl_platform.pxd

@@ -21,10 +21,7 @@
     SYCL platform-related helper functions.
 """
 
-from ._backend cimport (
-    DPCTLSyclPlatformRef,
-    DPCTLSyclDeviceSelectorRef,
-)
+from ._backend cimport DPCTLSyclDeviceSelectorRef, DPCTLSyclPlatformRef
 
 
 cdef class _SyclPlatform:

dpctl/_sycl_platform.pyx

@@ -21,11 +21,11 @@
 """
 
 from __future__ import print_function
-from ._backend cimport(
-    _backend_type,
+
+from ._backend cimport (
     DPCTLCString_Delete,
-    DPCTLFilterSelector_Create,
     DPCTLDeviceSelector_Delete,
+    DPCTLFilterSelector_Create,
     DPCTLPlatform_Copy,
     DPCTLPlatform_Create,
     DPCTLPlatform_CreateFromSelector,
@@ -43,7 +43,9 @@ from ._backend cimport(
     DPCTLSyclBackendType,
     DPCTLSyclDeviceSelectorRef,
     DPCTLSyclPlatformRef,
+    _backend_type,
 )
+
 from . import backend_type
 
 __all__ = [

dpctl/_sycl_queue.pxd

@@ -20,16 +20,14 @@
 """ This file declares the SyclQueue extension type.
 """
 
-from ._backend cimport (
-    DPCTLSyclDeviceRef,
-    DPCTLKernelArgType,
-    DPCTLSyclQueueRef,
-)
+from libcpp cimport bool as cpp_bool
+
+from ._backend cimport DPCTLKernelArgType, DPCTLSyclDeviceRef, DPCTLSyclQueueRef
 from ._sycl_context cimport SyclContext
-from ._sycl_event cimport SyclEvent
 from ._sycl_device cimport SyclDevice
+from ._sycl_event cimport SyclEvent
 from .program._program cimport SyclKernel
-from libcpp cimport bool as cpp_bool
+
 
 cdef void default_async_error_handler(int) nogil except *