Merge pull request #97 from automl/bug/status_running

Fix non-existent 'RUNNING' status for runs
automl · Dec 18, 2023 · ea8554c · ea8554c
2 parents 64e4e9e + 73e477a
commit ea8554c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,8 @@
+# Version 1.1.4
+
+## Bug-Fixes
+- Don't convert BOHB runs with status 'running' (consistent with SMAC).
+
 # Version 1.1.3
 
 ## Bug-Fixes

diff --git a/deepcave/runs/converters/bohb.py b/deepcave/runs/converters/bohb.py
@@ -49,7 +49,6 @@ def from_path(cls, path):
 
         first_starttime = None
         for bohb_run in bohb.get_all_runs():
-
             times = bohb_run.time_stamps
             starttime = times["started"]
             endtime = times["finished"]
@@ -77,7 +76,7 @@ def from_path(cls, path):
             if "SUCCESS" in status or "TERMINATED" in status or "COMPLETED" in status:
                 status = Status.SUCCESS
             elif "RUNNING" in status or "QUEUED" in status or "REVIEW" in status:
-                status = Status.RUNNING
+                continue
             else:
                 status = Status.CRASHED