fal 1.16.1__tar.gz → 1.17.1__tar.gz

{fal-1.16.1/fal.egg-info → fal-1.17.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fal
-Version: 1.16.1
+Version: 1.17.1
 Summary: fal is an easy-to-use Serverless Python Framework
 Author: Features & Labels <support@fal.ai>
 Requires-Python: >=3.8

{fal-1.16.1 → fal-1.17.1/fal.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fal
-Version: 1.16.1
+Version: 1.17.1
 Summary: fal is an easy-to-use Serverless Python Framework
 Author: Features & Labels <support@fal.ai>
 Requires-Python: >=3.8

{fal-1.16.1 → fal-1.17.1}/src/fal/_fal_version.py

@@ -17,5 +17,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '1.16.1'
-__version_tuple__ = version_tuple = (1, 16, 1)
+__version__ = version = '1.17.1'
+__version_tuple__ = version_tuple = (1, 17, 1)

{fal-1.16.1 → fal-1.17.1}/src/fal/files.py

@@ -28,26 +28,39 @@ class FalFileSystem(AbstractFileSystem):
             },
         )
 
+    def _ls(self, path):
+        response = self._client.get(f"/files/list/{path}")
+        response.raise_for_status()
+        files = response.json()
+        return sorted(
+            (
+                {
+                    "name": entry["path"],
+                    "size": entry["size"],
+                    "type": "file" if entry["is_file"] else "directory",
+                    "mtime": entry["updated_time"],
+                }
+                for entry in files
+            ),
+            key=lambda x: x["name"],
+        )
+
     def ls(self, path, detail=True, **kwargs):
-        if path in self.dircache:
-            entries = self.dircache[path]
+        abs_path = "/" + path.lstrip("/")
+        if abs_path in self.dircache:
+            entries = self.dircache[abs_path]
+        elif abs_path in ["/", "", "."]:
+            entries = [
+                {
+                    "name": "/data",
+                    "size": 0,
+                    "type": "directory",
+                    "mtime": 0,
+                }
+            ]
         else:
-            response = self._client.get(f"/files/list/{path.lstrip('/')}")
-            response.raise_for_status()
-            files = response.json()
-            entries = sorted(
-                (
-                    {
-                        "name": entry["path"].lstrip("/data/"),
-                        "size": entry["size"],
-                        "type": "file" if entry["is_file"] else "directory",
-                        "mtime": entry["updated_time"],
-                    }
-                    for entry in files
-                ),
-                key=lambda x: x["name"],
-            )
-        self.dircache[path] = entries
+            entries = self._ls(abs_path)
+        self.dircache[abs_path] = entries
 
         if detail:
             return entries
@@ -68,7 +81,7 @@ class FalFileSystem(AbstractFileSystem):
             return
 
         with open(lpath, "wb") as fobj:
-            response = self._client.get(f"/files/file/{rpath.lstrip('/')}")
+            response = self._client.get(f"/files/file/{rpath}")
             response.raise_for_status()
             fobj.write(response.content)
 
@@ -78,13 +91,13 @@ class FalFileSystem(AbstractFileSystem):
 
         with open(lpath, "rb") as fobj:
             response = self._client.post(
-                f"/files/file/local/{rpath.lstrip('/')}",
+                f"/files/file/local/{rpath}",
                 files={"file_upload": (posixpath.basename(lpath), fobj, "text/plain")},
             )
             response.raise_for_status()
         self.dircache.clear()
 
     def rm(self, path, **kwargs):
-        response = self._client.delete(f"/files/file/{path.lstrip('/')}")
+        response = self._client.delete(f"/files/file/{path}")
         response.raise_for_status()
         self.dircache.clear()

{fal-1.16.1 → fal-1.17.1}/src/fal/toolkit/utils/download_utils.py

@@ -6,6 +6,7 @@ import os
 import shutil
 import subprocess
 import sys
+from contextlib import suppress
 from pathlib import Path, PurePath
 from tempfile import NamedTemporaryFile, TemporaryDirectory
 from urllib.parse import urlparse
@@ -451,8 +452,11 @@ def clone_repository(
             with TemporaryDirectory(
                 dir=target_dir, suffix=f"{local_repo_path.name}.tmp.old"
             ) as tmp_dir:
-                os.rename(local_repo_path, tmp_dir)
-                shutil.rmtree(tmp_dir)
+                with suppress(FileNotFoundError):
+                    # repository might be already deleted by another worker
+                    os.rename(local_repo_path, tmp_dir)
+                    # sometimes seeing FileNotFoundError even here on juicefs
+                    shutil.rmtree(tmp_dir)
 
     # NOTE: using the target_dir to be able to avoid potential copies across temp fs
     # and target fs, and also to be able to atomically rename repo_name dir into place