Merge pull request #228 from dos-group/behind-threshold

Increasing Real-Time Factor Offset "Fix"

vessim/_util.py

@@ -25,10 +25,12 @@ def to_simtime(self, dt: datetime) -> int:
 def disable_rt_warnings(behind_threshold: float):
     """Disables Mosaik's rt_check warnings.
 
-    The simulation is always behind by a few fractions of a second (which is
-    fine, code needs time to execute) which Mosaik logs as a Warning. These
-    Warnings are flagged as bugs in Mosaik's current developement and should be
-    fixed within its next release. Until then, this function should do.
+    Mosaik's current implementation of the real-time checks are faulty and a new
+    implementation is already outlined in
+    https://gitlab.com/mosaik/mosaik/-/issues/172. As this issue was opened in
+    June 2023, it is unlikely that the issue will be resolved soon. Therefore,
+    we disable the warnings with a threshold after which these offsets are
+    logged.
 
     Args:
         behind_threshold: Time the simulation is allowed to be behind schedule.

vessim/cosim.py

@@ -145,7 +145,7 @@ def run(
         until: Optional[int] = None,
         rt_factor: Optional[float] = None,
         print_progress: bool | Literal["individual"] = True,
-        behind_threshold: float = 0.01,
+        behind_threshold: float = float("inf"),
     ):
         if until is None:
             # there is no integer representing infinity in python