update timeout hadnling - define default value only once

* definition in OMSessionABC
* move `set_timeout()` to this class
* all other places use `timeout: Optional[float] = None`

Author: syntron

OMPython/OMCSession.py

@@ -927,7 +927,7 @@ class OMSessionABC(metaclass=OMSessionMeta):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             **kwargs,
     ) -> None:
         """
@@ -939,7 +939,8 @@ def __init__(
         self.model_execution_local = False
 
         # store variables
-        self._timeout = timeout
+        self._timeout = 10.0
+        self.set_timeout(timeout=timeout)
         # command prefix (to be used for docker or WSL)
         self._cmd_prefix: list[str] = []
 
@@ -948,6 +949,19 @@ def __post_init__(self) -> None:
         Post initialisation method.
         """
 
+    def set_timeout(self, timeout: Optional[float] = None) -> float:
+        """
+        Set the timeout to be used for OMC communication (OMCSession).
+
+        The defined value is set and the current value is returned. If None is provided as argument, nothing is changed.
+        """
+        retval = self._timeout
+        if timeout is not None:
+            if timeout <= 0.0:
+                raise OMCSessionException(f"Invalid timeout value: {timeout}!")
+            self._timeout = timeout
+        return retval
+
     def get_cmd_prefix(self) -> list[str]:
         """
         Get session definition used for this instance of OMPath.
@@ -1040,7 +1054,7 @@ class OMCSessionABC(OMSessionABC, metaclass=abc.ABCMeta):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             **kwargs,
     ) -> None:
         """
@@ -1156,19 +1170,6 @@ def _timeout_loop(
             yield True
         yield False
 
-    def set_timeout(self, timeout: Optional[float] = None) -> float:
-        """
-        Set the timeout to be used for OMC communication (OMCSession).
-
-        The defined value is set and the current value is returned. If None is provided as argument, nothing is changed.
-        """
-        retval = self._timeout
-        if timeout is not None:
-            if timeout <= 0.0:
-                raise OMCSessionException(f"Invalid timeout value: {timeout}!")
-            self._timeout = timeout
-        return retval
-
     @staticmethod
     def escape_str(value: str) -> str:
         """
@@ -1432,7 +1433,7 @@ class OMCSessionPort(OMCSessionABC):
     def __init__(
             self,
             omc_port: str,
-            timeout: float = 10.0,
+            timeout: Optional[float] = None,
     ) -> None:
         super().__init__(timeout=timeout)
         self._omc_port = omc_port
@@ -1445,7 +1446,7 @@ class OMCSessionLocal(OMCSessionABC):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             omhome: Optional[str | os.PathLike] = None,
     ) -> None:
 
@@ -1526,7 +1527,7 @@ class OMCSessionZMQ(OMSessionABC):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             omhome: Optional[str] = None,
             omc_process: Optional[OMCSessionABC] = None,
     ) -> None:
@@ -1597,7 +1598,7 @@ class OMCSessionDockerABC(OMCSessionABC, metaclass=abc.ABCMeta):
 
     def __init__(
             self,
-            timeout: float = 10.0,
+            timeout: Optional[float] = None,
             docker: Optional[str] = None,
             dockerContainer: Optional[str] = None,
             dockerExtraArgs: Optional[list] = None,
@@ -1752,7 +1753,7 @@ class OMCSessionDocker(OMCSessionDockerABC):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             docker: Optional[str] = None,
             dockerExtraArgs: Optional[list] = None,
             dockerOpenModelicaPath: str | os.PathLike = "omc",
@@ -1905,7 +1906,7 @@ class OMCSessionDockerContainer(OMCSessionDockerABC):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             dockerContainer: Optional[str] = None,
             dockerExtraArgs: Optional[list] = None,
             dockerOpenModelicaPath: str | os.PathLike = "omc",
@@ -2005,7 +2006,7 @@ class OMCSessionWSL(OMCSessionABC):
 
     def __init__(
             self,
-            timeout: float = 10.00,
+            timeout: Optional[float] = None,
             wsl_omc: str = 'omc',
             wsl_distribution: Optional[str] = None,
             wsl_user: Optional[str] = None,
@@ -2093,7 +2094,7 @@ class OMSessionRunner(OMSessionABC):
 
     def __init__(
             self,
-            timeout: float = 10.0,
+            timeout: Optional[float] = None,
             version: str = "1.27.0",
             ompath_runner: Type[OMPathRunnerABC] = OMPathRunnerLocal,
             cmd_prefix: Optional[list[str]] = None,