fal 1.27.0__tar.gz → 1.28.0__tar.gz

{fal-1.27.0/fal.egg-info → fal-1.28.0}/PKG-INFO

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

{fal-1.27.0 → fal-1.28.0/fal.egg-info}/PKG-INFO

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

{fal-1.27.0 → fal-1.28.0}/src/fal/_fal_version.py

@@ -17,5 +17,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '1.27.0'
-__version_tuple__ = version_tuple = (1, 27, 0)
+__version__ = version = '1.28.0'
+__version_tuple__ = version_tuple = (1, 28, 0)

{fal-1.27.0 → fal-1.28.0}/src/fal/cli/_utils.py

@@ -12,7 +12,7 @@ def get_client(host: str, team: str | None = None):
 
 def is_app_name(app_ref: tuple[str, str | None]) -> bool:
     is_single_file = app_ref[1] is None
-    is_python_file = app_ref[0].endswith(".py")
+    is_python_file = app_ref[0] is None or app_ref[0].endswith(".py")
 
     return is_single_file and not is_python_file
 
@@ -42,9 +42,16 @@ def get_app_data_from_toml(app_name):
 
     app_auth = app_data.pop("auth", "private")
     app_deployment_strategy = app_data.pop("deployment_strategy", "recreate")
-    app_no_scale = app_data.pop("no_scale", False)
+
+    if "no_scale" in app_data:
+        # Deprecated
+        app_no_scale = app_data.pop("no_scale")
+        print("[WARNING] no_scale is deprecated, use app_scale_settings instead")
+        app_reset_scale = not app_no_scale
+    else:
+        app_reset_scale = app_data.pop("app_scale_settings", False)
 
     if len(app_data) > 0:
         raise ValueError(f"Found unexpected keys in pyproject.toml: {app_data}")
 
-    return app_ref, app_auth, app_deployment_strategy, app_no_scale
+    return app_ref, app_auth, app_deployment_strategy, app_reset_scale

{fal-1.27.0 → fal-1.28.0}/src/fal/cli/deploy.py

@@ -66,9 +66,9 @@ def _deploy_from_reference(
     app_ref: Tuple[Optional[Union[Path, str]], ...],
     app_name: str,
     args,
-    auth: Optional[Literal["public", "shared", "private"]] = None,
-    deployment_strategy: Optional[Literal["recreate", "rolling"]] = None,
-    scale: bool = True,
+    auth: Optional[Literal["public", "shared", "private"]],
+    deployment_strategy: Optional[Literal["recreate", "rolling"]],
+    scale: bool,
 ):
     from fal.api import FalServerlessError, FalServerlessHost
     from fal.utils import load_function_from
@@ -146,7 +146,7 @@ def _deploy(args):
             raise ValueError("Cannot use --app-name or --auth with app name reference.")
 
         app_name = args.app_ref[0]
-        app_ref, app_auth, app_deployment_strategy, app_no_scale = (
+        app_ref, app_auth, app_deployment_strategy, app_scale_settings = (
             get_app_data_from_toml(app_name)
         )
         file_path, func_name = RefAction.split_ref(app_ref)
@@ -157,7 +157,7 @@ def _deploy(args):
         app_name = args.app_name
         app_auth = args.auth
         app_deployment_strategy = args.strategy
-        app_no_scale = args.no_scale
+        app_scale_settings = args.app_scale_settings
 
     _deploy_from_reference(
         (file_path, func_name),
@@ -165,7 +165,7 @@ def _deploy(args):
         args,
         app_auth,
         app_deployment_strategy,
-        scale=not app_no_scale,
+        scale=app_scale_settings,
     )
 
 
@@ -233,12 +233,17 @@ def add_parser(main_subparsers, parents):
     )
     parser.add_argument(
         "--no-scale",
+        action="store_false",
+        dest="app_scale_settings",
+        default=False,
+        help="Use the previous deployment of the application for scale settings. "
+        "This is the default behavior.",
+    )
+    parser.add_argument(
+        "--reset-scale",
         action="store_true",
-        help=(
-            "Use min_concurrency/max_concurrency/concurrency_buffer/max_multiplexing "
-            "from previous deployment of application with this name, if exists. "
-            "Otherwise will use the values from the application code."
-        ),
+        dest="app_scale_settings",
+        help="Use the application code for scale settings.",
     )
 
     parser.set_defaults(func=_deploy)

{fal-1.27.0 → fal-1.28.0}/src/fal/cli/profile.py

@@ -68,12 +68,17 @@ def _unset(args, config: Config | None = None):
 def _key_set(args):
     config = Config(validate_profile=True)
 
+    args.console.print("Go to https://fal.ai/dashboard/keys to create a key.")
+    args.console.print(
+        "Enter the key in the format [cyan][bold]id:secret[/][/]. You can copy and paste it from the browser."  # noqa: E501
+    )
+
     while True:
         key = input("Enter the key: ")
         if ":" in key:
             break
         args.console.print(
-            "[red]Invalid key. The key must be in the format [bold]key:value[/].[/]"
+            "[yellow]Invalid key. The key must be in the format:[/] [cyan][bold]id:secret[/][/]."  # noqa: E501
         )
 
     with config.edit():
@@ -99,10 +104,10 @@ def _create(args):
     with config.edit() as config:
         config._config[args.PROFILE] = {}
 
-    args.console.print(f"Profile [cyan]{args.PROFILE}[/] created.")
     args.console.print(
-        f"Use [bold]fal profile set {args.PROFILE}[/] to set it as default."
+        f"Profile [cyan]{args.PROFILE}[/] created. Setting it as default profile..."
     )
+    _set(args)
 
 
 def _delete(args):
@@ -184,7 +189,7 @@ def add_parser(main_subparsers, parents):
     )
     host_set_parser.set_defaults(func=_host_set)
 
-    create_help = "Create a new profile. Use 'fal profile set <name>' to set it as default after creation."  # noqa: E501
+    create_help = "Create a new profile."
     create_parser = subparsers.add_parser(
         "create",
         description=create_help,

{fal-1.27.0 → fal-1.28.0}/src/fal/config.py

@@ -6,7 +6,9 @@ from typing import Dict, Iterator, List, Optional
 
 SETTINGS_SECTION = "__internal__"
 
-NO_PROFILE_ERROR = ValueError("No profile set.")
+NO_PROFILE_ERROR = ValueError(
+    "No profile set. Use command [bold]fal profile set[/] to set a profile."
+)
 
 
 class Config:

{fal-1.27.0 → fal-1.28.0}/src/fal/exceptions/auth.py

@@ -7,5 +7,5 @@ class UnauthenticatedException(FalServerlessException):
     def __init__(self) -> None:
         super().__init__(
             "You must be authenticated. "
-            "Login via `fal auth login` or make sure to setup fal keys correctly."
+            "Use [bold]fal auth login[/] or [bold]fal profile key[/] to set your fal key."  # noqa: E501
         )

{fal-1.27.0 → fal-1.28.0}/src/fal/toolkit/utils/download_utils.py

@@ -6,6 +6,7 @@ import os
 import shutil
 import subprocess
 import sys
+import time
 from contextlib import suppress
 from pathlib import Path, PurePath
 from tempfile import NamedTemporaryFile, TemporaryDirectory
@@ -345,6 +346,14 @@ def _stream_url_data_to_file(
         raise DownloadError("Received less data than expected from the server.")
 
 
+def _mark_used_dir(dir: Path):
+    used_file = dir / ".fal_used"
+    day_ago = time.time() - 86400
+    if not used_file.exists() or used_file.stat().st_mtime < day_ago:
+        # Touch a last-used file to indicate that the weights have been used
+        used_file.touch()
+
+
 def download_model_weights(
     url: str, force: bool = False, request_headers: dict[str, str] | None = None
 ) -> Path:
@@ -375,19 +384,34 @@ def download_model_weights(
 
     if weights_dir.exists() and not force:
         try:
-            weights_path = next(weights_dir.glob("*"))
+            # TODO: sometimes the directory can hold multiple files
+            # Example:
+            # .fal/model_weights/00155dc2d9579360d577d1a87d31b52c21135c14a5f44fcbab36fbb8352f3e0d  # noqa: E501
+            # We need to either not allow multiple files in the directory or
+            # find the one that is the most recently used.
+            weights_path = next(
+                # Ignore .fal dotfiles since they are metadata files
+                f
+                for f in weights_dir.glob("*")
+                if not f.name.startswith(".fal")
+            )
+            _mark_used_dir(weights_dir)
             return weights_path
         # The model weights directory is empty, so we need to download the weights
         except StopIteration:
             pass
 
-    return download_file(
+    path = download_file(
         url,
         target_dir=weights_dir,
         force=force,
         request_headers=request_headers,
     )
 
+    _mark_used_dir(weights_dir)
+
+    return path
+
 
 def clone_repository(
     https_url: str,

{fal-1.27.0 → fal-1.28.0}/tests/cli/test_deploy.py

@@ -39,7 +39,7 @@ def mock_args(
     app_name: Optional[str] = None,
     auth: Optional[str] = None,
     strategy: Optional[str] = None,
-    no_scale: bool = False,
+    reset_scale: bool = False,
 ):
     args = MagicMock()
 
@@ -47,8 +47,7 @@ def mock_args(
     args.app_name = app_name
     args.auth = auth
     args.strategy = strategy
-
-    args.no_scale = no_scale
+    args.app_scale_settings = reset_scale
 
     return args
 
@@ -75,7 +74,7 @@ def test_deploy_with_toml_success(
         args,
         "shared",
         "rolling",
-        scale=True,
+        scale=False,
     )
 
 
@@ -101,7 +100,7 @@ def test_deploy_with_toml_no_auth(
         args,
         "private",
         "recreate",
-        scale=True,
+        scale=False,
     )
 
 
@@ -212,7 +211,7 @@ def test_deploy_with_toml_deployment_strategy(
         args,
         "shared",
         "rolling",
-        scale=True,
+        scale=False,
     )
 
 
@@ -236,7 +235,7 @@ def test_deploy_with_toml_default_deployment_strategy(
         args,
         "private",
         "recreate",
-        scale=True,
+        scale=False,
     )
 
 
@@ -258,7 +257,7 @@ def test_deploy_with_cli_auth(
         args,
         "shared",
         None,
-        scale=True,
+        scale=False,
     )
 
 
@@ -280,19 +279,19 @@ def test_deploy_with_cli_deployment_strategy(
         args,
         None,
         "rolling",
-        scale=True,
+        scale=False,
     )
 
 
 @patch("fal.cli._utils.find_pyproject_toml", return_value="pyproject.toml")
 @patch("fal.cli._utils.parse_pyproject_toml")
 @patch("fal.cli.deploy._deploy_from_reference")
-def test_deploy_with_cli_no_scale(
+def test_deploy_with_cli_reset_scale(
     mock_deploy_ref, mock_parse_toml, mock_find_toml, mock_parse_pyproject_toml
 ):
     mock_parse_toml.return_value = mock_parse_pyproject_toml
 
-    args = mock_args(app_ref=("src/my_app/inference.py", "MyApp"), no_scale=True)
+    args = mock_args(app_ref=("src/my_app/inference.py", "MyApp"), reset_scale=True)
 
     _deploy(args)
 
@@ -302,7 +301,7 @@ def test_deploy_with_cli_no_scale(
         args,
         None,
         None,
-        scale=False,
+        scale=True,
     )
 
 
@@ -324,5 +323,5 @@ def test_deploy_with_cli_scale(
         args,
         None,
         None,
-        scale=True,
+        scale=False,
     )