outsource convert_to_str_paths

python/podio/root_io.py

@@ -1,11 +1,10 @@
 #!/usr/bin/env python3
 """Python module for reading root files containing podio Frames"""
 
-from collections.abc import Iterable
-import os
-from pathlib import Path
 from ROOT import gSystem
 
+from utils import convert_to_str_paths
+
 gSystem.Load("libpodioRootIO")  # noqa: E402
 from ROOT import podio  # noqa: E402 # pylint: disable=wrong-import-position
 
@@ -17,24 +16,6 @@
 )
 
 
-def convert_to_str_paths(filenames):
-    """Converts filenames to string paths, handling both string and pathlib.Path objects and
-       iterables of such objects.
-
-    Args:
-        filenames (str, Path, or Iterable[str | Path]): A single filepath or an iterable of
-        filepaths to convert to str object(s).
-
-    Returns:
-        list[str]: A list of filepaths as strings.
-    """
-
-    if isinstance(filenames, Iterable) and not isinstance(filenames, (str, Path)):
-        return [os.fspath(fn) for fn in filenames]
-
-    return [os.fspath(filenames)]
-
-
 class Reader(BaseReaderMixin):
     """Reader class for reading podio root files."""
 

python/podio/sio_io.py

@@ -2,7 +2,8 @@
 """Python module for reading sio files containing podio Frames"""
 
 from ROOT import gSystem
-from root_io import convert_to_str_paths
+
+from utils import convert_to_str_paths
 
 if gSystem.DynamicPathName("libpodioSioIO.so", True):
     gSystem.Load("libpodioSioIO")  # noqa: 402

python/podio/utils.py

@@ -0,0 +1,21 @@
+import os
+from collections.abc import Iterable
+from pathlib import Path
+
+
+def convert_to_str_paths(filenames):
+    """Converts filenames to string paths, handling both string and pathlib.Path objects and
+       iterables of such objects.
+
+    Args:
+        filenames (str, Path, or Iterable[str | Path]): A single filepath or an iterable of
+        filepaths to convert to str object(s).
+
+    Returns:
+        list[str]: A list of filepaths as strings.
+    """
+
+    if isinstance(filenames, Iterable) and not isinstance(filenames, (str, Path)):
+        return [os.fspath(fn) for fn in filenames]
+
+    return [os.fspath(filenames)]