Use tuple for default ROI_table_names value in copy_ome_zarr (ref #448)

Author: tcompa

fractal_tasks_core/__FRACTAL_MANIFEST__.json

@@ -254,6 +254,10 @@
           },
           "ROI_table_names": {
             "title": "Roi Table Names",
+            "default": [
+              "FOV_ROI_table",
+              "well_ROI_table"
+            ],
             "type": "array",
             "items": {
               "type": "string"

fractal_tasks_core/tasks/copy_ome_zarr.py

@@ -18,7 +18,6 @@
 from pathlib import Path
 from typing import Any
 from typing import Dict
-from typing import Optional
 from typing import Sequence
 
 import anndata as ad
@@ -46,7 +45,7 @@ def copy_ome_zarr(
     metadata: Dict[str, Any],
     project_to_2D: bool = True,
     suffix: str = "mip",
-    ROI_table_names: Optional[Sequence[str]] = None,
+    ROI_table_names: tuple[str, ...] = ("FOV_ROI_table", "well_ROI_table"),
 ) -> Dict[str, Any]:
 
     """
@@ -111,10 +110,6 @@ def copy_ome_zarr(
         # FIXME create a standard suffix (with timestamp)
         raise NotImplementedError
 
-    if ROI_table_names is None:
-        # FIXME: are these defaults OK?
-        ROI_table_names = ["FOV_ROI_table", "well_ROI_table"]
-
     # List all plates
     in_path = Path(input_paths[0])
     list_plates = [p.as_posix() for p in Path(in_path).glob("*.zarr")]