Bug fixing the I/O operations for mrCOSTS

Author: klapo

pydmd/mrcosts.py

@@ -575,14 +575,16 @@ def to_netcdf(self, filename, filepath="."):
 
         """
         for c in self._costs_array:
+            fname = ".".join(
+                (
+                    filename,
+                    f"window={c.window_length:}",
+                    "nc",
+                )
+            )
+            fpath = os.path.join(filepath, fname)
             c.to_xarray().to_netcdf(
-                filepath.join(
-                    (
-                        filename,
-                        f"window={c.window_length:}",
-                        "nc",
-                    )
-                ),
+                fpath,
                 engine="h5netcdf",
                 invalid_netcdf=True,
             )

tests/test_mrcosts.py

@@ -293,7 +293,7 @@ def test_netcdf(tmp_path):
     """
     # Move the I/O tests to the temporary test directory.
     mrc.to_netcdf("tests", filepath=tmp_path)
-    file_list = glob.glob("*tests*.nc")
+    file_list = glob.glob(os.path.join(tmp_path, "*tests*.nc"))
     mrc_from_file = mrCOSTS()
     mrc_from_file.from_netcdf(file_list)