fixed linter issues

Author: georgevigelette

src/openlifu/io/LIFUHVController.py

@@ -988,62 +988,62 @@ def get_vmon_values(self) -> list[dict]:
             raise  # Re-raise the exception for the caller to handle
 
     def set_raw_dac(self, dac_id: int = 0, dac_value: int = 0) -> int:
-            """
-            Set Raw DAC value.
-
-            Args:
-                dac_id (int): The desired DAC to set (default is 0). Valid IDs are 0, 1, 2, and 3.
-                dac_value (int): The desired DAC value (default is 0). Must be between 0 and 4095.
-            Returns:
-                int: The current output DAC value.
-
-            Raises:
-                ValueError: If the controller is not connected.
-            """
-            if not self.uart.is_connected():
-                raise ValueError("High voltage controller not connected")
-
-            if dac_id not in [0, 1, 2 ,3]:
-                raise ValueError("Invalid DAC ID. Must be 0, 1, 2, or 3")
-
-            if dac_value not in range(4096):
-                raise ValueError("Invalid DAC value. Must be 0 to 4095")
-
-            try:
-                if self.uart.demo_mode:
-                    return dac_value
-                logger.info("Setting Raw DAC value.")
-                data = bytes(
-                    [
-                        (dac_value >> 8) & 0xFF,  # High byte (most significant bits)
-                        dac_value & 0xFF,  # Low byte (least significant bits)
-                    ]
-                )
-                r = self.uart.send_packet(
-                    id=None,
-                    addr=dac_id,
-                    packetType=OW_POWER,
-                    command=OW_POWER_RAW_DAC,
-                    data=data,
-                )
-
-                self.uart.clear_buffer()
-                # r.print_packet()
-
-                if r.packet_type == OW_ERROR:
-                    logger.error("Error setting DAC value")
-                    return -1
-
-                logger.info(f"Set DAC value to {dac_value}")
+        """
+        Set Raw DAC value.
+
+        Args:
+            dac_id (int): The desired DAC to set (default is 0). Valid IDs are 0, 1, 2, and 3.
+            dac_value (int): The desired DAC value (default is 0). Must be between 0 and 4095.
+        Returns:
+            int: The current output DAC value.
+
+        Raises:
+            ValueError: If the controller is not connected.
+        """
+        if not self.uart.is_connected():
+            raise ValueError("High voltage controller not connected")
+
+        if dac_id not in [0, 1, 2 ,3]:
+            raise ValueError("Invalid DAC ID. Must be 0, 1, 2, or 3")
+
+        if dac_value not in range(4096):
+            raise ValueError("Invalid DAC value. Must be 0 to 4095")
+
+        try:
+            if self.uart.demo_mode:
                 return dac_value
+            logger.info("Setting Raw DAC value.")
+            data = bytes(
+                [
+                    (dac_value >> 8) & 0xFF,  # High byte (most significant bits)
+                    dac_value & 0xFF,  # Low byte (least significant bits)
+                ]
+            )
+            r = self.uart.send_packet(
+                id=None,
+                addr=dac_id,
+                packetType=OW_POWER,
+                command=OW_POWER_RAW_DAC,
+                data=data,
+            )
 
-            except ValueError as v:
-                logger.error("ValueError: %s", v)
-                raise  # Re-raise the exception for the caller to handle
+            self.uart.clear_buffer()
+            # r.print_packet()
 
-            except Exception as e:
-                logger.error("Unexpected error during process: %s", e)
-                raise  # Re-raise the exception for the caller to handle
+            if r.packet_type == OW_ERROR:
+                logger.error("Error setting DAC value")
+                return -1
+
+            logger.info(f"Set DAC value to {dac_value}")
+            return dac_value
+
+        except ValueError as v:
+            logger.error("ValueError: %s", v)
+            raise  # Re-raise the exception for the caller to handle
+
+        except Exception as e:
+            logger.error("Unexpected error during process: %s", e)
+            raise  # Re-raise the exception for the caller to handle
 
     def hv_enable(self, enable: bool = False) -> bool:
         """

src/openlifu/io/LIFUUart.py

@@ -314,11 +314,11 @@ def reopen_after_reset(self, retries=5, delay=1.0):
                         self.connect()
                         log.info("Reconnected to UART on %s after reset.", self.port)
                         return True
-                    except Exception as e:
+                    except (OSError, ValueError) as e:
                         log.warning("Reconnect attempt failed: %s", e)
             log.error("Failed to reconnect UART after reset.")
             return False
-        except Exception as e:
+        except (OSError, ValueError) as e:
             log.error("reopen_after_reset() error: %s", e)
             return False