openrunner-sdk 2.37.35__tar.gz → 2.37.36__tar.gz

{openrunner_sdk-2.37.35 → openrunner_sdk-2.37.36}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openrunner-sdk
-Version: 2.37.35
+Version: 2.37.36
 Summary: OpenRunner SDK - W&B-compatible ML experiment tracking client
 Project-URL: Homepage, https://github.com/jqueguiner/openrunner
 Project-URL: Repository, https://github.com/jqueguiner/openrunner

{openrunner_sdk-2.37.35 → openrunner_sdk-2.37.36}/openrunner/__init__.py

@@ -181,7 +181,7 @@ launch.from_run = _launch_from_run  # type: ignore[attr-defined]
 # openrunner.trace.patch_openai() syntax
 trace.patch_openai = _patch_openai  # type: ignore[attr-defined]
 
-__version__ = "2.37.34"
+__version__ = "2.37.36"
 
 logger = logging.getLogger("openrunner")
 

{openrunner_sdk-2.37.35 → openrunner_sdk-2.37.36}/openrunner/run.py

@@ -921,7 +921,7 @@ class Run:
 
             # Get artifact by project lookup
             art_result = self._client._client.get(
-                f"/projects/{self._project}/artifacts"
+                f"/projects/{self._project_ref()}/artifacts"
             )
             if art_result.status_code == 200:
                 arts = art_result.json()
@@ -940,10 +940,25 @@ class Run:
 
         return result
 
+    def _project_ref(self) -> str:
+        """Path segment for ``/projects/{ref}/*`` endpoints.
+
+        Those endpoints are keyed by the project **UUID**, but a run may have
+        been constructed with a project *name* (``org/name``). Prefer the
+        server-resolved UUID (``self._project_id``, populated from the
+        create_run response) and fall back to the name only when it is absent
+        (e.g. offline mode). Using the bare name against a UUID-keyed endpoint
+        404s -- the bug that made ``set_model_card``/``get_model_card`` and
+        artifact-alias resolution silently no-op.
+        """
+        return getattr(self, "_project_id", None) or self._project
+
     def _resolve_artifact_id(self, name: str) -> str | None:
         """Look up an artifact's id by name within this run's project."""
         try:
-            resp = self._client._client.get(f"/projects/{self._project}/artifacts")
+            resp = self._client._client.get(
+                f"/projects/{self._project_ref()}/artifacts"
+            )
             if resp.status_code == 200:
                 for a in resp.json():
                     if a.get("name") == name:

{openrunner_sdk-2.37.35 → openrunner_sdk-2.37.36}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "openrunner-sdk"
-version = "2.37.35"
+version = "2.37.36"
 description = "OpenRunner SDK - W&B-compatible ML experiment tracking client"
 readme = "README.md"
 license = {text = "MIT"}

{openrunner_sdk-2.37.35 → openrunner_sdk-2.37.36}/tests/test_run_module.py

@@ -729,6 +729,35 @@ def test_resolve_artifact_id_exception():
     assert run._resolve_artifact_id("m") is None
 
 
+def test_resolve_artifact_id_uses_project_uuid():
+    # Regression: the /projects/{ref}/artifacts endpoint is keyed by the
+    # project UUID, not the project name. When the server-resolved UUID is
+    # available it must be used, else the lookup 404s and set_model_card
+    # silently no-ops.
+    run = _bare_run(project="gladia/openpii")
+    run._project_id = "96de4897-38b0-4327-a07b-1c5a71871ef9"
+    run._client._client = MagicMock()
+    resp = MagicMock(status_code=200)
+    resp.json.return_value = [{"id": "aid", "name": "m"}]
+    run._client._client.get.return_value = resp
+    assert run._resolve_artifact_id("m") == "aid"
+    run._client._client.get.assert_called_once_with(
+        "/projects/96de4897-38b0-4327-a07b-1c5a71871ef9/artifacts"
+    )
+
+
+def test_resolve_artifact_id_falls_back_to_name_without_uuid():
+    # Offline / pre-create: no UUID yet -> fall back to the project name so
+    # behavior is unchanged when _project_id is absent.
+    run = _bare_run(project="gladia/openpii")  # _bare_run sets no _project_id
+    run._client._client = MagicMock()
+    resp = MagicMock(status_code=200)
+    resp.json.return_value = [{"id": "aid", "name": "m"}]
+    run._client._client.get.return_value = resp
+    assert run._resolve_artifact_id("m") == "aid"
+    run._client._client.get.assert_called_once_with("/projects/openpii/artifacts")
+
+
 def test_set_model_card_success():
     run = _bare_run()
     run._client._client = MagicMock()