fal 1.5.9__tar.gz → 1.5.11__tar.gz

{fal-1.5.9/fal.egg-info → fal-1.5.11}/PKG-INFO

@@ -1,12 +1,12 @@
 Metadata-Version: 2.1
 Name: fal
-Version: 1.5.9
+Version: 1.5.11
 Summary: fal is an easy-to-use Serverless Python Framework
 Author: Features & Labels <support@fal.ai>
 Requires-Python: >=3.8
 Description-Content-Type: text/markdown
 Requires-Dist: isolate[build]<0.15.0,>=0.14.2
-Requires-Dist: isolate-proto==0.5.5
+Requires-Dist: isolate-proto<0.7.0,>=0.6.0
 Requires-Dist: grpcio==1.64.0
 Requires-Dist: dill==0.3.7
 Requires-Dist: cloudpickle==3.0.0

{fal-1.5.9 → fal-1.5.11/fal.egg-info}/PKG-INFO

@@ -1,12 +1,12 @@
 Metadata-Version: 2.1
 Name: fal
-Version: 1.5.9
+Version: 1.5.11
 Summary: fal is an easy-to-use Serverless Python Framework
 Author: Features & Labels <support@fal.ai>
 Requires-Python: >=3.8
 Description-Content-Type: text/markdown
 Requires-Dist: isolate[build]<0.15.0,>=0.14.2
-Requires-Dist: isolate-proto==0.5.5
+Requires-Dist: isolate-proto<0.7.0,>=0.6.0
 Requires-Dist: grpcio==1.64.0
 Requires-Dist: dill==0.3.7
 Requires-Dist: cloudpickle==3.0.0

{fal-1.5.9 → fal-1.5.11}/fal.egg-info/requires.txt

@@ -1,5 +1,5 @@
 isolate[build]<0.15.0,>=0.14.2
-isolate-proto==0.5.5
+isolate-proto<0.7.0,>=0.6.0
 grpcio==1.64.0
 dill==0.3.7
 cloudpickle==3.0.0

{fal-1.5.9 → fal-1.5.11}/pyproject.toml

@@ -23,7 +23,7 @@ readme = "README.md"
 requires-python = ">=3.8"
 dependencies = [
     "isolate[build]>=0.14.2,<0.15.0",
-    "isolate-proto==0.5.5",
+    "isolate-proto>=0.6.0,<0.7.0",
     "grpcio==1.64.0",
     "dill==0.3.7",
     "cloudpickle==3.0.0",

{fal-1.5.9 → fal-1.5.11}/src/fal/_fal_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '1.5.9'
-__version_tuple__ = version_tuple = (1, 5, 9)
+__version__ = version = '1.5.11'
+__version_tuple__ = version_tuple = (1, 5, 11)

{fal-1.5.9 → fal-1.5.11}/src/fal/app.py

@@ -69,6 +69,12 @@ async def _set_logger_labels(
     logger_labels: dict[str, str], channel: async_grpc.Channel
 ):
     try:
+        import sys
+
+        # Flush any prints that were buffered before setting the logger labels
+        sys.stderr.flush()
+        sys.stdout.flush()
+
         isolate = definitions.IsolateStub(channel)
         isolate_request = definitions.SetMetadataRequest(
             # TODO: when submit is shipped, get task_id from an env var
@@ -77,9 +83,11 @@ async def _set_logger_labels(
         )
         res = isolate.SetMetadata(isolate_request)
         code = await res.code()
-        assert str(code) == "StatusCode.OK"
+        assert str(code) == "StatusCode.OK", str(code)
     except BaseException:
-        logger.debug("Failed to set logger labels", exc_info=True)
+        # NOTE hiding this for now to not print on every request
+        # logger.debug("Failed to set logger labels", exc_info=True)
+        pass
 
 
 def wrap_app(cls: type[App], **kwargs) -> fal.api.IsolatedFunction:

{fal-1.5.9 → fal-1.5.11}/src/fal/apps.py

@@ -173,7 +173,8 @@ def submit(app_id: str, arguments: dict[str, Any], *, path: str = "") -> Request
     app_id = _backwards_compatible_app_id(app_id)
     url = _QUEUE_URL_FORMAT.format(app_id=app_id)
     if path:
-        url += "/" + path.removeprefix("/")
+        _path = path[len("/") :] if path.startswith("/") else path
+        url += "/" + _path
 
     creds = get_default_credentials()
 
@@ -235,7 +236,8 @@ def _connect(app_id: str, *, path: str = "/realtime") -> Iterator[_RealtimeConne
     app_id = _backwards_compatible_app_id(app_id)
     url = _REALTIME_URL_FORMAT.format(app_id=app_id)
     if path:
-        url += "/" + path.removeprefix("/")
+        _path = path[len("/") :] if path.startswith("/") else path
+        url += "/" + _path
 
     creds = get_default_credentials()
 

{fal-1.5.9 → fal-1.5.11}/src/fal/workflows.py

@@ -5,7 +5,7 @@ import webbrowser
 from argparse import ArgumentParser
 from collections import Counter
 from dataclasses import dataclass, field
-from typing import Any, Iterator, Union, cast
+from typing import Any, Dict, Iterator, List, Union, cast
 
 import graphlib
 import rich
@@ -21,8 +21,8 @@ from fal import flags
 from fal.exceptions import FalServerlessException
 from fal.rest_client import REST_CLIENT
 
-JSONType = Union[dict[str, Any], list[Any], str, int, float, bool, None, "Leaf"]
-SchemaType = dict[str, Any]
+JSONType = Union[Dict[str, Any], List[Any], str, int, float, bool, None, "Leaf"]
+SchemaType = Dict[str, Any]
 
 VARIABLE_PREFIX = "$"
 INPUT_VARIABLE_NAME = "input"
@@ -50,7 +50,13 @@ def parse_leaf(raw_leaf: str) -> Leaf:
             f"Invalid leaf: {raw_leaf} (must start with a reference)"
         )
 
-    leaf: Leaf = ReferenceLeaf(reference.removeprefix(VARIABLE_PREFIX))
+    # remove the $ prefix
+    _reference = (
+        reference[len(VARIABLE_PREFIX) :]
+        if reference.startswith(VARIABLE_PREFIX)
+        else reference
+    )
+    leaf: Leaf = ReferenceLeaf(_reference)
     for raw_part in raw_parts:
         if raw_part.isdigit():
             leaf = IndexLeaf(leaf, int(raw_part))

{fal-1.5.9 → fal-1.5.11}/tests/cli/test_deploy.py

@@ -1,4 +1,4 @@
-from typing import Optional
+from typing import Optional, Tuple
 from unittest.mock import MagicMock, patch
 
 import pytest
@@ -30,7 +30,7 @@ def mock_parse_pyproject_toml():
 
 
 def mock_args(
-    app_ref: tuple[str],
+    app_ref: Tuple[str, Optional[str]],
     app_name: Optional[str] = None,
     auth: Optional[str] = None,
     strategy: Optional[str] = None,

{fal-1.5.9 → fal-1.5.11}/tests/cli/test_run.py

@@ -1,3 +1,4 @@
+from typing import Optional, Tuple
 from unittest.mock import MagicMock, patch
 
 import pytest
@@ -34,7 +35,7 @@ def mocked_fal_serverless_host(host):
     return mock
 
 
-def mock_args(host, func_ref: tuple[str]):
+def mock_args(host, func_ref: Tuple[str, Optional[str]]):
     args = MagicMock()
     args.host = host
     args.func_ref = func_ref

{fal-1.5.9 → fal-1.5.11}/tests/test_apps.py

@@ -5,7 +5,7 @@ import subprocess
 import time
 from contextlib import contextmanager, suppress
 from datetime import datetime
-from typing import Generator
+from typing import Generator, List, Tuple
 
 import fal
 import fal.api as api
@@ -92,7 +92,7 @@ def container_addition_app(input: Input) -> Output:
 
 @fal.function(
     keep_alive=300,
-    requirements=["fastapi", "uvicorn", "pydantic==1.10.12"],
+    requirements=["fastapi", "uvicorn", "pydantic==1.10.18"],
     machine_type="S",
     max_concurrency=1,
     max_multiplexing=30,
@@ -224,7 +224,7 @@ class RTOutput(BaseModel):
 
 
 class RTOutputs(BaseModel):
-    texts: list[str]
+    texts: List[str]
 
 
 class RealtimeApp(fal.App, keep_alive=300, max_concurrency=1):
@@ -252,7 +252,7 @@ class RealtimeApp(fal.App, keep_alive=300, max_concurrency=1):
 
 
 @pytest.fixture(scope="module")
-def aliased_app() -> Generator[tuple[str, str], None, None]:
+def aliased_app() -> Generator[Tuple[str, str], None, None]:
     # Create a temporary app, register it, and return the ID of it.
 
     import uuid
@@ -418,6 +418,7 @@ def test_stateful_app_client(test_stateful_app: str):
     assert response["result"] == 0
 
 
+@pytest.mark.flaky(max_runs=3)
 def test_app_cancellation(test_app: str, test_cancellable_app: str):
     request_handle = apps.submit(
         test_cancellable_app, arguments={"lhs": 1, "rhs": 2, "wait_time": 10}
@@ -516,7 +517,7 @@ def test_404_response(test_app: str, request: pytest.FixtureRequest):
         apps.run(test_app, path="/other", arguments={"lhs": 1, "rhs": 2})
 
 
-def test_app_deploy_scale(aliased_app: tuple[str, str]):
+def test_app_deploy_scale(aliased_app: Tuple[str, str]):
     import uuid
     from dataclasses import replace
 
@@ -558,7 +559,7 @@ def test_app_deploy_scale(aliased_app: tuple[str, str]):
         assert found.max_multiplexing == 30
 
 
-def test_app_update_app(aliased_app: tuple[str, str]):
+def test_app_update_app(aliased_app: Tuple[str, str]):
     app_revision, app_alias = aliased_app
 
     host: api.FalServerlessHost = addition_app.host  # type: ignore
@@ -608,7 +609,7 @@ def test_app_update_app(aliased_app: tuple[str, str]):
         assert res.max_multiplexing == new_max_multiplexing
 
 
-def test_app_set_delete_alias(aliased_app: tuple[str, str]):
+def test_app_set_delete_alias(aliased_app: Tuple[str, str]):
     app_revision, app_alias = aliased_app
 
     host: api.FalServerlessHost = addition_app.host  # type: ignore