fix erros when genome folder is an integer

kbase · Jul 2, 2024 · 325caaf · 325caaf
1 parent 827bf2c
commit 325caaf
Show file tree

Hide file tree

Showing 6 changed files with 27 additions and 3 deletions.
diff --git a/src/loaders/compute_tools/checkm2/versions.yaml b/src/loaders/compute_tools/checkm2/versions.yaml
@@ -26,4 +26,9 @@ versions:
       - Remove the 'threads' and 'program_threads' parameters 
         and introduce the capability to specify the number of threads per tool run
       - pass job_id to the tool container and remove node_id
+    reference_db_version: 1.0.1
+  - version: 0.1.5
+    date: 2024-06-25
+    notes: |
+      - Bug fix: tool_common.py - Converted Data IDs to string format to ensure proper comparison with associated folder names
     reference_db_version: 1.0.1
diff --git a/src/loaders/compute_tools/eggnog/versions.yaml b/src/loaders/compute_tools/eggnog/versions.yaml
@@ -15,4 +15,10 @@ versions:
       - Remove the 'threads' and 'program_threads' parameters 
         and introduce the capability to specify the number of threads per tool run
       - pass job_id to the tool container and remove node_id
+    reference_db_version: 5.0.2
+
+  - version: 0.1.3
+    date: 2024-06-25
+    notes: |
+      - Bug fix: tool_common.py - Converted Data IDs to string format to ensure proper comparison with associated folder names
     reference_db_version: 5.0.2
diff --git a/src/loaders/compute_tools/gtdb_tk/versions.yaml b/src/loaders/compute_tools/gtdb_tk/versions.yaml
@@ -35,4 +35,9 @@ versions:
       - Remove the 'threads' and 'program_threads' parameters 
         and introduce the capability to specify the number of threads per tool run
       - pass job_id to the tool container and remove node_id
+    reference_db_version: release214
+  - version: 0.1.6
+    date: 2024-06-25
+    notes: |
+      - Bug fix: tool_common.py - Converted Data IDs to string format to ensure proper comparison with associated folder names
     reference_db_version: release214
diff --git a/src/loaders/compute_tools/mash/versions.yaml b/src/loaders/compute_tools/mash/versions.yaml
@@ -12,4 +12,8 @@ versions:
     notes: |
       - Remove the 'threads' and 'program_threads' parameters 
         and introduce the capability to specify the number of threads per tool run
-      - pass job_id to the tool container and remove node_id
+      - pass job_id to the tool container and remove node_id
+  - version: 0.1.4
+    date: 2024-06-25
+    notes: |
+      - Bug fix: tool_common.py - Converted Data IDs to string format to ensure proper comparison with associated folder names
diff --git a/src/loaders/compute_tools/microtrait/versions.yaml b/src/loaders/compute_tools/microtrait/versions.yaml
@@ -18,4 +18,8 @@ versions:
     notes: |
       - Remove the 'threads' and 'program_threads' parameters 
         and introduce the capability to specify the number of threads per tool run
-      - pass job_id to the tool container and remove node_id
+      - pass job_id to the tool container and remove node_id
+  - version: 0.1.5
+    date: 2024-06-25
+    notes: |
+      - Bug fix: tool_common.py - Converted Data IDs to string format to ensure proper comparison with associated folder names
diff --git a/src/loaders/compute_tools/tool_common.py b/src/loaders/compute_tools/tool_common.py
@@ -211,7 +211,7 @@ def _get_data_ids(self):
             with self._data_id_file:
                 df = pd.read_csv(self._data_id_file, sep='\t')
                 try:
-                    data_ids = df[loader_common_names.DATA_ID_COLUMN_HEADER]
+                    data_ids = df[loader_common_names.DATA_ID_COLUMN_HEADER].astype(str).tolist()  # convert to string in case of int directory names
                 except KeyError:
                     raise ValueError(
                         f"Please ensure {loader_common_names.DATA_ID_COLUMN_HEADER} column exists in the "