code-loader 1.0.187.dev0__tar.gz → 1.0.188__tar.gz

{code_loader-1.0.187.dev0 → code_loader-1.0.188}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: code-loader
-Version: 1.0.187.dev0
+Version: 1.0.188
 Summary: 
 License: MIT
 Author: dorhar

{code_loader-1.0.187.dev0 → code_loader-1.0.188}/code_loader/inner_leap_binder/leapbinder.py

@@ -1,6 +1,6 @@
 
 import inspect
-from typing import Callable, List, Optional, Dict, Any, Type, Union
+from typing import Callable, List, Optional, Dict, Any, Type, Union, get_args
 
 import numpy as np
 import numpy.typing as npt
@@ -32,21 +32,17 @@ mapping_runtime_mode_env_var_mame = '__MAPPING_RUNTIME_MODE__'
 
 
 def _stringized_annotation_type_name(annotation: Any) -> Optional[str]:
-    """Return the bare type name when ``annotation`` is a *stringized* annotation
-    (e.g. produced by ``from __future__ import annotations`` or a quoted hint),
-    otherwise ``None``. code_loader inspects raw annotations expecting real classes,
-    so a stringized annotation silently breaks type detection."""
+    """Bare type name if ``annotation`` is a stringized annotation (from ``from __future__
+    import annotations`` or a quoted hint), else ``None``. code_loader inspects raw
+    annotations expecting real classes, so stringization silently breaks type detection."""
     if isinstance(annotation, str):
         return annotation.split("[")[0].strip().rsplit(".", 1)[-1]
     return None
 
 
-def _reject_stringized_sample_preprocess_response(function: Callable, arg_name: str, annotation: Any) -> None:
-    """Fail fast when a ``SamplePreprocessResponse`` argument was stringized. Otherwise
-    ``arg_type == SamplePreprocessResponse`` silently evaluates to ``False`` and the
-    argument is mis-wired as a regular input — a bug that only surfaces on the platform.
-    Fires only on the actual damage (a stringized SamplePreprocessResponse), not merely
-    because the file uses future annotations."""
+def _reject_stringized_sample_preprocess_response(function: Callable[..., Any], arg_name: str, annotation: Any) -> None:
+    """Fail fast on a stringized SamplePreprocessResponse arg (otherwise it silently
+    mis-wires as a regular input, surfacing only on the platform)."""
     if _stringized_annotation_type_name(annotation) == SamplePreprocessResponse.__name__:
         raise Exception(
             f"Argument '{arg_name}' of function '{function.__name__}' is annotated with a string "
@@ -170,8 +166,8 @@ class LeapBinder:
 
         if visualizer_type.value not in map_leap_data_type_to_visualizer_class:
             raise Exception(
-                f'The visualizer_type is invalid. current visualizer_type: {visualizer_type}, '  # type: ignore[attr-defined]
-                f'should be one of : {", ".join([arg.__name__ for arg in LeapData.__args__])}')
+                f'The visualizer_type is invalid. current visualizer_type: {visualizer_type}, '
+                f'should be one of : {", ".join([arg.__name__ for arg in get_args(LeapData)])}')
 
         func_annotations = function.__annotations__
         if "return" not in func_annotations:
@@ -188,10 +184,10 @@ class LeapBinder:
                     f"'from __future__ import annotations' (or a quoted return hint), which stringizes "
                     f"annotations and prevents Tensorleap from detecting the visualizer type. Remove "
                     f"that import (or the quotes) so the return type is a real class.")
-            if return_type not in LeapData.__args__:  # type: ignore[attr-defined]
+            if return_type not in get_args(LeapData):
                 raise Exception(
-                    f'The return type of function {function.__name__} is invalid. current return type: {return_type}, '  # type: ignore[attr-defined]
-                    f'should be one of : {", ".join([arg.__name__ for arg in LeapData.__args__])}')
+                    f'The return type of function {function.__name__} is invalid. current return type: {return_type}, '
+                    f'should be one of : {", ".join([arg.__name__ for arg in get_args(LeapData)])}')
 
             expected_return_type = map_leap_data_type_to_visualizer_class[visualizer_type.value]
             if not issubclass(return_type, expected_return_type):

{code_loader-1.0.187.dev0 → code_loader-1.0.188}/code_loader/inner_leap_binder/leapbinder_decorators.py

@@ -223,6 +223,59 @@ def _add_mapping_connections(connects_to, arg_names, node_mapping_type, name):
         _add_mapping_connection(user_unique_name, connection_destinations, arg_names, name, node_mapping_type)
 
 
+def _require_sample_preprocess_response_supplied(user_function: Callable, args: tuple, kwargs: dict) -> None:
+    """A SamplePreprocessResponse argument is auto-injected by the platform / check_dataset
+    but NOT inside integration_test, where the author calls the function directly. Fail fast
+    with an actionable message instead of a raw 'missing argument' TypeError."""
+    spr_arg_name = None
+    for arg_name, arg_type in inspect.getfullargspec(user_function).annotations.items():
+        if arg_type == SamplePreprocessResponse:
+            spr_arg_name = arg_name
+            break
+    if spr_arg_name is None:
+        return
+    signature = inspect.signature(user_function)
+    param = signature.parameters.get(spr_arg_name)
+    # Only enforce a REQUIRED SamplePreprocessResponse arg. If it has a default it is
+    # optional and the function may legitimately be called with or without it.
+    if param is None or param.default is not inspect.Parameter.empty:
+        return
+    try:
+        supplied = signature.bind_partial(*args, **kwargs).arguments
+    except TypeError:
+        return
+    if spr_arg_name not in supplied:
+        raise Exception(
+            f"'{user_function.__name__}' takes a SamplePreprocessResponse argument "
+            f"'{spr_arg_name}', but it was not provided. Inside integration_test pass it "
+            f"explicitly, e.g. {user_function.__name__}(..., "
+            f"SamplePreprocessResponse(sample_id, preprocess)).")
+
+
+def _warn_loss_inputs_not_wired_to_model() -> None:
+    """Warn when a custom loss consumes an input encoder the model is not given.
+
+    Such an input stays a plain NodeMappingType.Input (model-fed inputs get re-tagged to
+    Input0/Input1/...) and crashes the platform model build. Losses only — the same pattern
+    in a metric or visualizer builds fine."""
+    loss_types = {NodeMappingType.Loss, NodeMappingType.CustomLoss}
+    for connection in leap_binder.mapping_connections:
+        if connection.node is None or connection.node.type not in loss_types:
+            continue
+        for arg_name, source in (connection.node_inputs or {}).items():
+            if source is not None and source.type == NodeMappingType.Input:
+                input_name = source.name or arg_name
+                store_general_warning(
+                    key=("loss_input_not_wired_to_model", connection.node.name, input_name),
+                    message=(
+                        f"Custom loss '{connection.node.name}' consumes input '{input_name}', "
+                        f"which is not fed to the model. This is not supported: the platform builds "
+                        f"losses into the model graph, so an input the model does not take has no "
+                        f"source there and the model build fails. Instead, carry this data in the "
+                        f"ground truth and derive it inside the loss (not as a separate input "
+                        f"encoder)."))
+
+
 def tensorleap_integration_test():
     def decorating_function(integration_test_function: Callable):
         leap_binder.integration_test_func = integration_test_function
@@ -279,6 +332,8 @@ def tensorleap_integration_test():
             finally:
                 _called_from_inside_tl_integration_test_decorator = False
 
+            _warn_loss_inputs_not_wired_to_model()
+
             leap_binder.check()
 
         return inner
@@ -822,6 +877,7 @@ def tensorleap_custom_metric(name: str,
             _called_from_inside_tl_decorator += 1
 
             try:
+                _require_sample_preprocess_response_supplied(user_function, args, kwargs)
                 result = user_function(*args, **kwargs)
             finally:
                 _called_from_inside_tl_decorator -= 1
@@ -1031,6 +1087,7 @@ def tensorleap_custom_instances_metric(name: str,
             _called_from_inside_tl_decorator += 1
 
             try:
+                _require_sample_preprocess_response_supplied(user_function, args, kwargs)
                 result = user_function(*args, **kwargs)
             finally:
                 _called_from_inside_tl_decorator -= 1
@@ -1383,6 +1440,7 @@ def tensorleap_custom_visualizer(name: str, visualizer_type: LeapDataType,
             _called_from_inside_tl_decorator += 1
 
             try:
+                _require_sample_preprocess_response_supplied(user_function, args, kwargs)
                 result = user_function(*args, **kwargs)
             finally:
                 _called_from_inside_tl_decorator -= 1
@@ -1929,8 +1987,13 @@ def tensorleap_input_encoder(name: str, channel_dim=_UNSET, model_input_index=No
 
                 )
 
+        # channel_dim is the channel axis in the BATCHED tensor (batch = axis 0):
+        # 1 for channel-first (NCHW), -1 for channel-last (NHWC); 0 is invalid.
         if channel_dim <= 0 and channel_dim != -1:
-            raise Exception(f"Channel dim for input {name} is expected to be either -1 or positive")
+            raise Exception(
+                f"Invalid channel_dim ({channel_dim}) for input '{name}'. channel_dim is the "
+                f"channel axis in the batched tensor (batch = axis 0), so 0 is invalid. Use 1 "
+                f"for channel-first (NCHW) and -1 for channel-last (NHWC).")
 
         def _validate_input_args(sample_id: Union[int, str], preprocess_response: PreprocessResponse):
             assert type(sample_id) == preprocess_response.sample_id_type, \
@@ -2149,6 +2212,7 @@ def tensorleap_custom_loss(name: str, connects_to=None):
             _called_from_inside_tl_decorator += 1
 
             try:
+                _require_sample_preprocess_response_supplied(user_function, args, kwargs)
                 result = user_function(*args, **kwargs)
             finally:
                 _called_from_inside_tl_decorator -= 1

{code_loader-1.0.187.dev0 → code_loader-1.0.188}/pyproject.toml

@@ -1,7 +1,7 @@
 [tool.poetry]
 name = "code-loader"
 
-version = "1.0.187.dev0"
+version = "1.0.188"
 description = ""
 authors = ["dorhar <doron.harnoy@tensorleap.ai>"]
 license = "MIT"