Refactor logger initialization to support custom names

Added an optional `logger_name` parameter to the logger's constructor. This allows setting a custom logger name, defaulting to the class name if none is provided. Improves flexibility for log management in different contexts.

Author: felixschndr

inverter

@@ -53,13 +53,14 @@ from inverter import Inverter
 from logger import LoggerMixin
 from tibber_api_handler import TibberAPIHandler
 
-logger = LoggerMixin()
+logger = LoggerMixin("Bashscript")
 logger.write_newlines_to_log_file()
 logger.log.info(f"Inverter is manually controlled by the user {getpass.getuser()}: {sys.argv}")
 
 if sys.argv[1] == "--status":
     try:
-        inverter = Inverter(True)
+        inverter = Inverter()
+        inverter.log.name += " User"
         soc = inverter.get_state_of_charge(True)
         mode = inverter.get_operation_mode(True).name
         print(f"SoC: {soc}\nmode: {mode}")
@@ -70,7 +71,8 @@ if sys.argv[1] == "--status":
 if sys.argv[1] == "--mode":
     try:
         new_mode = OperationMode[sys.argv[2]]
-        inverter = Inverter(True)
+        inverter = Inverter()
+        inverter.log.name += " User"
         last_mode = inverter.get_operation_mode(True).name
         inverter.set_operation_mode(new_mode)
         print(f"last mode: {last_mode}\nnew mode: {new_mode.name}")

source/inverter.py

@@ -6,24 +6,16 @@
 from goodwe import inverter as GoodweInverter
 from goodwe.et import OperationMode
 from logger import LoggerMixin
-from sems_portal_api_handler import SemsPortalApiHandler
 
 
 class Inverter(LoggerMixin):
-    def __init__(self, controlled_by_bash_script: bool = False):
+    def __init__(self):
         super().__init__()
 
         self._device = None
         self.hostname = EnvironmentVariableGetter.get("INVERTER_HOSTNAME")
         self.battery_capacity = EnergyAmount(int(EnvironmentVariableGetter.get("INVERTER_BATTERY_CAPACITY")))
 
-        self.sems_portal_api_handler = SemsPortalApiHandler()
-
-        # Add a notice to the loggers name to make to easier to identify actions taken by a user manually
-        if controlled_by_bash_script:
-            self.log.name += " USER"
-            self.sems_portal_api_handler.log.name += " USER"
-
     @property
     def device(self) -> GoodweInverter:
         """

source/logger.py

@@ -32,12 +32,14 @@ class LoggerMixin:
     Notice that the loglevel trace with a weight of 5 was added.
     """
 
-    def __init__(self):
+    def __init__(self, logger_name: str = None):
         root_logger = logging.getLogger()
         if len(root_logger.handlers) == 0:
             self._set_logger(root_logger)
 
-        self.log = logging.getLogger(self.__class__.__name__)
+        if not logger_name:
+            logger_name = self.__class__.__name__
+        self.log = logging.getLogger(logger_name)
 
     def _set_logger(self, root_logger: logging.Logger) -> None:
         """