Merge pull request #150 from lumalabs/release-please--branches--main--changes--next

release: 1.7.3

Author: karanganesan

.github/workflows/ci.yml

@@ -12,7 +12,6 @@ jobs:
   lint:
     name: lint
     runs-on: ubuntu-latest
-
     steps:
       - uses: actions/checkout@v4
 
@@ -33,7 +32,6 @@ jobs:
   test:
     name: test
     runs-on: ubuntu-latest
-
     steps:
       - uses: actions/checkout@v4
 

.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.7.2"
+  ".": "1.7.3"
 }

CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 1.7.3 (2025-04-12)
+
+Full Changelog: [v1.7.2...v1.7.3](https://github.com/lumalabs/lumaai-python/compare/v1.7.2...v1.7.3)
+
+### Bug Fixes
+
+* **perf:** optimize some hot paths ([ae01192](https://github.com/lumalabs/lumaai-python/commit/ae01192924c54d4a2a7d9a70d0df88e209163989))
+* **perf:** skip traversing types for NotGiven values ([43ac98d](https://github.com/lumalabs/lumaai-python/commit/43ac98d293f3e373ac986a4cabd036a54b1e8213))
+
+
+### Chores
+
+* **internal:** expand CI branch coverage ([a006432](https://github.com/lumalabs/lumaai-python/commit/a00643219c4eebcfe34eca5046e5380a96945257))
+* **internal:** reduce CI branch coverage ([7c6570e](https://github.com/lumalabs/lumaai-python/commit/7c6570e1e1a9cd62d67017d62af8375b8eb6a145))
+* **internal:** slight transform perf improvement ([#149](https://github.com/lumalabs/lumaai-python/issues/149)) ([b95d415](https://github.com/lumalabs/lumaai-python/commit/b95d415fb6f3a92b90e78044c3756b84611a9179))
+* **tests:** improve enum examples ([#151](https://github.com/lumalabs/lumaai-python/issues/151)) ([dc358d1](https://github.com/lumalabs/lumaai-python/commit/dc358d17dc810dc8e1de849679a618c61a626cd5))
+
 ## 1.7.2 (2025-04-04)
 
 Full Changelog: [v1.7.1...v1.7.2](https://github.com/lumalabs/lumaai-python/compare/v1.7.1...v1.7.2)

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "lumaai"
-version = "1.7.2"
+version = "1.7.3"
 description = "The official Python library for the lumaai API"
 dynamic = ["readme"]
 license = "Apache-2.0"

src/lumaai/_utils/_transform.py

@@ -5,13 +5,15 @@
 import pathlib
 from typing import Any, Mapping, TypeVar, cast
 from datetime import date, datetime
-from typing_extensions import Literal, get_args, override, get_type_hints
+from typing_extensions import Literal, get_args, override, get_type_hints as _get_type_hints
 
 import anyio
 import pydantic
 
 from ._utils import (
     is_list,
+    is_given,
+    lru_cache,
     is_mapping,
     is_iterable,
 )
@@ -108,6 +110,7 @@ class Params(TypedDict, total=False):
     return cast(_T, transformed)
 
 
+@lru_cache(maxsize=8096)
 def _get_annotated_type(type_: type) -> type | None:
     """If the given type is an `Annotated` type then it is returned, if not `None` is returned.
 
@@ -142,6 +145,10 @@ def _maybe_transform_key(key: str, type_: type) -> str:
     return key
 
 
+def _no_transform_needed(annotation: type) -> bool:
+    return annotation == float or annotation == int
+
+
 def _transform_recursive(
     data: object,
     *,
@@ -184,6 +191,15 @@ def _transform_recursive(
             return cast(object, data)
 
         inner_type = extract_type_arg(stripped_type, 0)
+        if _no_transform_needed(inner_type):
+            # for some types there is no need to transform anything, so we can get a small
+            # perf boost from skipping that work.
+            #
+            # but we still need to convert to a list to ensure the data is json-serializable
+            if is_list(data):
+                return data
+            return list(data)
+
         return [_transform_recursive(d, annotation=annotation, inner_type=inner_type) for d in data]
 
     if is_union_type(stripped_type):
@@ -245,6 +261,11 @@ def _transform_typeddict(
     result: dict[str, object] = {}
     annotations = get_type_hints(expected_type, include_extras=True)
     for key, value in data.items():
+        if not is_given(value):
+            # we don't need to include `NotGiven` values here as they'll
+            # be stripped out before the request is sent anyway
+            continue
+
         type_ = annotations.get(key)
         if type_ is None:
             # we do not have a type annotation for this field, leave it as is
@@ -332,6 +353,15 @@ async def _async_transform_recursive(
             return cast(object, data)
 
         inner_type = extract_type_arg(stripped_type, 0)
+        if _no_transform_needed(inner_type):
+            # for some types there is no need to transform anything, so we can get a small
+            # perf boost from skipping that work.
+            #
+            # but we still need to convert to a list to ensure the data is json-serializable
+            if is_list(data):
+                return data
+            return list(data)
+
         return [await _async_transform_recursive(d, annotation=annotation, inner_type=inner_type) for d in data]
 
     if is_union_type(stripped_type):
@@ -393,10 +423,25 @@ async def _async_transform_typeddict(
     result: dict[str, object] = {}
     annotations = get_type_hints(expected_type, include_extras=True)
     for key, value in data.items():
+        if not is_given(value):
+            # we don't need to include `NotGiven` values here as they'll
+            # be stripped out before the request is sent anyway
+            continue
+
         type_ = annotations.get(key)
         if type_ is None:
             # we do not have a type annotation for this field, leave it as is
             result[key] = value
         else:
             result[_maybe_transform_key(key, type_)] = await _async_transform_recursive(value, annotation=type_)
     return result
+
+
+@lru_cache(maxsize=8096)
+def get_type_hints(
+    obj: Any,
+    globalns: dict[str, Any] | None = None,
+    localns: Mapping[str, Any] | None = None,
+    include_extras: bool = False,
+) -> dict[str, Any]:
+    return _get_type_hints(obj, globalns=globalns, localns=localns, include_extras=include_extras)

src/lumaai/_utils/_typing.py

@@ -13,6 +13,7 @@
     get_origin,
 )
 
+from ._utils import lru_cache
 from .._types import InheritsGeneric
 from .._compat import is_union as _is_union
 
@@ -66,6 +67,7 @@ def is_type_alias_type(tp: Any, /) -> TypeIs[typing_extensions.TypeAliasType]:
 
 
 # Extracts T from Annotated[T, ...] or from Required[Annotated[T, ...]]
+@lru_cache(maxsize=8096)
 def strip_annotated_type(typ: type) -> type:
     if is_required_type(typ) or is_annotated_type(typ):
         return strip_annotated_type(cast(type, get_args(typ)[0]))

src/lumaai/_version.py

@@ -1,4 +1,4 @@
 # File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
 __title__ = "lumaai"
-__version__ = "1.7.2"  # x-release-please-version
+__version__ = "1.7.3"  # x-release-please-version

tests/api_resources/generations/test_image.py

@@ -25,7 +25,7 @@ def test_method_create(self, client: LumaAI) -> None:
     @parametrize
     def test_method_create_with_all_params(self, client: LumaAI) -> None:
         image = client.generations.image.create(
-            aspect_ratio="1:1",
+            aspect_ratio="16:9",
             callback_url="https://example.com",
             character_ref={"identity0": {"images": ["https://example.com"]}},
             generation_type="image",
@@ -82,7 +82,7 @@ async def test_method_create(self, async_client: AsyncLumaAI) -> None:
     @parametrize
     async def test_method_create_with_all_params(self, async_client: AsyncLumaAI) -> None:
         image = await async_client.generations.image.create(
-            aspect_ratio="1:1",
+            aspect_ratio="16:9",
             callback_url="https://example.com",
             character_ref={"identity0": {"images": ["https://example.com"]}},
             generation_type="image",

tests/api_resources/generations/test_video.py

@@ -25,7 +25,7 @@ def test_method_create(self, client: LumaAI) -> None:
     @parametrize
     def test_method_create_with_all_params(self, client: LumaAI) -> None:
         video = client.generations.video.create(
-            aspect_ratio="1:1",
+            aspect_ratio="16:9",
             callback_url="https://example.com",
             duration="5s",
             generation_type="video",
@@ -78,7 +78,7 @@ async def test_method_create(self, async_client: AsyncLumaAI) -> None:
     @parametrize
     async def test_method_create_with_all_params(self, async_client: AsyncLumaAI) -> None:
         video = await async_client.generations.video.create(
-            aspect_ratio="1:1",
+            aspect_ratio="16:9",
             callback_url="https://example.com",
             duration="5s",
             generation_type="video",

tests/api_resources/test_generations.py

@@ -28,7 +28,7 @@ def test_method_create(self, client: LumaAI) -> None:
     @parametrize
     def test_method_create_with_all_params(self, client: LumaAI) -> None:
         generation = client.generations.create(
-            aspect_ratio="1:1",
+            aspect_ratio="16:9",
             callback_url="https://example.com",
             duration="5s",
             generation_type="video",
@@ -287,7 +287,7 @@ async def test_method_create(self, async_client: AsyncLumaAI) -> None:
     @parametrize
     async def test_method_create_with_all_params(self, async_client: AsyncLumaAI) -> None:
         generation = await async_client.generations.create(
-            aspect_ratio="1:1",
+            aspect_ratio="16:9",
             callback_url="https://example.com",
             duration="5s",
             generation_type="video",