[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

docs/conf.py

@@ -6,6 +6,7 @@
 -- Project information -----------------------------------------------------
 https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information
 """
+
 import os
 import sys
 from re import Match, match

tests/__init__.py

@@ -1,4 +1,5 @@
 """Pytest package for testing uoshardware."""
+
 from dataclasses import dataclass
 
 

tests/conftest.py

@@ -1,4 +1,5 @@
 """Module for package test configuration, scope=session."""
+
 import pytest
 
 from uoshardware import Loading

tests/interface/conftest.py

@@ -1,4 +1,5 @@
 """Module used for fixture initialisation in interface tests."""
+
 import pytest
 
 

tests/interface/test_package.py

@@ -1,4 +1,5 @@
 """Module for testing the interface package."""
+
 from time import sleep
 
 import pytest

tests/test_abstractions.py

@@ -1,4 +1,5 @@
 """Unit tests for the `abstractions` module."""
+
 import pytest
 
 from tests import Packet

tests/test_api.py

@@ -1,4 +1,5 @@
 """Unit tests for the api module."""
+
 from inspect import signature
 
 import pytest

tests/test_devices.py

@@ -1,4 +1,5 @@
 """Tests for the devices module."""
+
 import pytest
 
 from uoshardware import UOSUnsupportedError

uoshardware/__init__.py

@@ -1,4 +1,5 @@
 """The high level interface for communicating with UOS devices."""
+
 import logging
 from enum import Enum
 

uoshardware/abstractions.py

@@ -1,4 +1,5 @@
 """Module defining the base class and static func for interfaces."""
+
 from abc import ABCMeta, abstractmethod
 from dataclasses import dataclass, field
 from datetime import datetime
@@ -182,12 +183,13 @@ class UOSInterface(metaclass=ABCMeta):
     def execute_instruction(self, packet: NPCPacket) -> ComResult:  # dead: disable
         """Abstract method for executing instructions on UOSInterfaces.
 
-        :param packet: A tuple containing the uint8 npc packet for the UOS instruction.
+        :param packet: A tuple containing the uint8 npc packet for the
+            UOS instruction.
         :returns: ComResult object.
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         :raises: UOSCommunicationError if there is a problem completing
-                the action.
+            the action.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride "
@@ -201,12 +203,13 @@ def read_response(
         """Read ACK and Data packets from a UOSInterface.
 
         :param expect_packets: How many packets including ACK to expect
-        :param timeout_s: The maximum time this function will wait for data.
+        :param timeout_s: The maximum time this function will wait for
+            data.
         :return: COM Result object.
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         :raises: UOSCommunicationError if there is a problem completing
-                the action.
+            the action.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride "
@@ -219,9 +222,9 @@ def hard_reset(self) -> ComResult:
 
         :return: COM Result object.
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         :raises: UOSCommunicationError if there is a problem completing
-                the action.
+            the action.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride "
@@ -233,9 +236,9 @@ def open(self):
         """Abstract method for opening a connection to a UOSInterface.
 
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         :raises: UOSCommunicationError if there is a problem completing
-                the action.
+            the action.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride " f"{UOSInterface.open.__name__} prototype."
@@ -246,9 +249,9 @@ def close(self):
         """Abstract method for closing a connection to a UOSInterface.
 
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         :raises: UOSCommunicationError if there is a problem completing
-                the action.
+            the action.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride " f"{UOSInterface.close.__name__} prototype."
@@ -260,7 +263,7 @@ def is_active(self) -> bool:
 
         :return: Success boolean.
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride " f"{UOSInterface.close.__name__} prototype."
@@ -273,7 +276,7 @@ def enumerate_devices() -> list:
 
         :return: A list of possible UOSInterfaces on the server.
         :raises: UOSUnsupportedError if the interface hasn't been built
-                correctly.
+            correctly.
         """
         raise UOSUnsupportedError(
             "UOSInterfaces must over-ride "

uoshardware/api.py

@@ -1,4 +1,5 @@
 """Provides the HAL layer for communicating with the hardware."""
+
 from uoshardware import (
     Loading,
     Persistence,
@@ -29,7 +30,7 @@ def enumerate_system_devices(  # dead: disable
     """Iterate through all interfaces and locates available devices.
 
     :param interface_filter: Interface enum to limit the search to a
-    single interface type.
+        single interface type.
     :return: A list of uosinterface objects.
     """
     system_devices = []
@@ -44,7 +45,8 @@ def enumerate_system_devices(  # dead: disable
 def get_device_definition(identity: str) -> Device | None:
     """Look up the system config dictionary for the defined device mappings.
 
-    :param identity: String containing the lookup key of the device in the dictionary.
+    :param identity: String containing the lookup key of the device in
+        the dictionary.
     :return: Device Object or None if not found
     """
     if identity is not None and hasattr(Devices, identity):
@@ -58,9 +60,9 @@ def get_device_definition(identity: str) -> Device | None:
 class UOSDevice:  # dead: disable
     """Class for high level object-orientated control of UOS devices.
 
-    :ivar identity: The type of device, this is must have a valid device in the config.
-    :ivar address: Compliant connection string for identifying the
-        device and interface.
+    :ivar identity: The type of device, this is must have a valid device
+    in the config. :ivar address: Compliant connection string for
+    identifying the     device and interface.
     """
 
     __device: Device  # Device definitions as parsed from a compatible ini.
@@ -165,10 +167,11 @@ def get_gpio_input(
         """Read a GPIO pins level from device and returns the value.
 
         :param pin: The numeric number of the pin as defined in the
-                dictionary for that device.
-        :param pull_up: Enable the internal pull-up resistor. Default is false.
+            dictionary for that device.
+        :param pull_up: Enable the internal pull-up resistor. Default is
+            false.
         :param volatility: How volatile should the command be, use
-                constants from uoshardware.
+            constants from uoshardware.
         :return: ComResult object.
         """
         result = self.__execute_instruction(
@@ -219,8 +222,9 @@ def reset_all_io(self, volatility=Persistence.RAM) -> ComResult:
         """Execute the reset IO at the defined volatility level.
 
         :param volatility: Where should the pins reset from, use
-                constants from uoshardware.
-        :return: ComResult object containing the result of the reset operation..
+            constants from uoshardware.
+        :return: ComResult object containing the result of the reset
+            operation..
         """
         return self.__execute_instruction(
             UOSFunctions.reset_all_io,
@@ -258,11 +262,13 @@ def __execute_instruction(
         """Execute a generic UOS function and get the result.
 
         :param function: The name of the function in the OOL.
-        :param instruction_data: device_functions from the LUT, payload ect.
-        :param retry: Allows the instruction to retry execution when fails.
+        :param instruction_data: device_functions from the LUT, payload
+            ect.
+        :param retry: Allows the instruction to retry execution when
+            fails.
         :return: ComResult object
         :raises: UOSUnsupportedError if function is not possible on the
-                loaded device.
+            loaded device.
         """
         if (
             function.name not in self.__device.functions_enabled
@@ -373,7 +379,8 @@ def get_compatible_pins(self, function: UOSFunction) -> set:
     def get_functions_enabled(self) -> dict:  # dead: disable
         """Return functions enabled for the device.
 
-        :return: Dictionary of function names to list of Persistence levels.
+        :return: Dictionary of function names to list of Persistence
+            levels.
         """
         return self.__device.functions_enabled
 

uoshardware/devices/__init__.py

@@ -1,4 +1,5 @@
 """Packages used to define Devices supported by the library."""
+
 from dataclasses import dataclass
 
 from uoshardware.abstractions import Device
@@ -9,9 +10,8 @@
 class Devices:
     """Names for supported hardware linking to the Device object used.
 
-    :cvar hwid_0: device: _ARDUINO_NANO_3
-    :cvar arduino_nano: device: _ARDUINO_NANO_3
-    :cvar arduino_uno: device: _ARDUINO_NANO_3
+    :cvar hwid_0: device: _ARDUINO_NANO_3 :cvar arduino_nano: device:
+    _ARDUINO_NANO_3 :cvar arduino_uno: device: _ARDUINO_NANO_3
     """
 
     # Lookup constants linking devices to importable names

uoshardware/devices/_arduino.py

@@ -1,4 +1,5 @@
 """Module contains definitions for arduino devices."""
+
 from uoshardware import Persistence
 from uoshardware.abstractions import Device, Pin, UOSFunctions
 from uoshardware.interface import Interface

uoshardware/interface/__init__.py

@@ -1,4 +1,5 @@
 """Package defining io interfaces used for NPC Comms."""
+
 from enum import Enum
 
 from uoshardware.interface.serial import Serial

uoshardware/interface/serial.py

@@ -1,4 +1,5 @@
 """Module defining the low level UOSImplementation for serial port devices."""
+
 import platform
 from time import sleep, time_ns
 
@@ -16,11 +17,11 @@
 class Serial(UOSInterface):
     """Pyserial class that handles reading / writing to ports.
 
-    :ivar _device: Holds the pyserial device once opened. None if not opened.
-    :ivar _connection: Holds the standard connection string
-        'Interface'|'OS Connection String.
-    :ivar _port: Holds the port class, none type if device not instantiated.
-    :ivar _kwargs: Additional keyword arguments as defined in the documentation.
+    :ivar _device: Holds the pyserial device once opened. None if not
+    opened. :ivar _connection: Holds the standard connection string
+    'Interface'|'OS Connection String. :ivar _port: Holds the port
+    class, none type if device not instantiated. :ivar _kwargs:
+    Additional keyword arguments as defined in the documentation.
     """
 
     _device = None
@@ -100,9 +101,10 @@ def close(self):
     def execute_instruction(self, packet: NPCPacket):
         """Build and execute a new instruction packet.
 
-        :param packet: A tuple containing the uint8 npc packet for the UOS instruction.
+        :param packet: A tuple containing the uint8 npc packet for the
+            UOS instruction.
         :return: Tuple containing a status boolean and index 0 and a
-                result-set dict at index 1.
+            result-set dict at index 1.
         """
         if self._device is None:
             raise UOSCommunicationError(
@@ -124,7 +126,8 @@ def read_response(self, expect_packets: int, timeout_s: float):
         """Read ACK and response packets from the serial device.
 
         :param expect_packets: How many packets including ACK to expect.
-        :param timeout_s: The maximum time this function will wait for data.
+        :param timeout_s: The maximum time this function will wait for
+            data.
         :return: ComResult object.
         """
         if self._device is None:
@@ -173,7 +176,7 @@ def hard_reset(self):
         """Manually drive the DTR line low to reset the device.
 
         :return: Tuple containing a status boolean and index 0 and a
-                result-set dict at index 1.
+            result-set dict at index 1.
         """
         if self._device is None:
             raise UOSCommunicationError("Connection must be open to hard reset device.")
@@ -206,7 +209,8 @@ def decode_and_capture(
         :param byte_index: The index of the last 'valid' byte found.
         :param byte_in: The current byte for inspection.
         :param packet: The current packet of validated bytes.
-        :return: Tuple containing the updated byte index and updated packet.
+        :return: Tuple containing the updated byte index and updated
+            packet.
         """
         if byte_index == -1:  # start symbol
             if byte_in == b">":

uoshardware/interface/stub.py

@@ -1,4 +1,5 @@
 """Package is used as a simulated UOSInterface for test purposes."""
+
 from uoshardware import UOSCommunicationError
 from uoshardware.abstractions import ComResult, NPCPacket, UOSInterface