arize-phoenix 8.19.1__tar.gz → 8.21.0__tar.gz

{arize_phoenix-8.19.1 → arize_phoenix-8.21.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arize-phoenix
-Version: 8.19.1
+Version: 8.21.0
 Summary: AI Observability and Evaluation
 Project-URL: Documentation, https://docs.arize.com/phoenix/
 Project-URL: Issues, https://github.com/Arize-ai/phoenix/issues
@@ -71,7 +71,7 @@ Requires-Dist: opentelemetry-sdk; extra == 'container'
 Requires-Dist: opentelemetry-semantic-conventions; extra == 'container'
 Requires-Dist: prometheus-client; extra == 'container'
 Requires-Dist: py-grpc-prometheus; extra == 'container'
-Requires-Dist: strawberry-graphql[opentelemetry]==0.262.0; extra == 'container'
+Requires-Dist: strawberry-graphql[opentelemetry]==0.262.5; extra == 'container'
 Requires-Dist: umap-learn; extra == 'container'
 Requires-Dist: uvloop; (platform_system != 'Windows') and extra == 'container'
 Provides-Extra: dev
@@ -102,7 +102,7 @@ Requires-Dist: pytest-postgresql; extra == 'dev'
 Requires-Dist: pytest-xdist; extra == 'dev'
 Requires-Dist: pytest==8.3.3; extra == 'dev'
 Requires-Dist: ruff==0.6.9; extra == 'dev'
-Requires-Dist: strawberry-graphql[debug-server,opentelemetry]==0.262.0; extra == 'dev'
+Requires-Dist: strawberry-graphql[debug-server,opentelemetry]==0.262.5; extra == 'dev'
 Requires-Dist: tabulate; extra == 'dev'
 Requires-Dist: tox-uv==1.11.3; extra == 'dev'
 Requires-Dist: tox==4.18.1; extra == 'dev'

{arize_phoenix-8.19.1 → arize_phoenix-8.21.0}/pyproject.toml

@@ -89,7 +89,7 @@ dev = [
   "pytest-postgresql",
   "asyncpg",
   "psycopg[binary,pool]",
-  "strawberry-graphql[debug-server,opentelemetry]==0.262.0",  # need to pin version because we're monkey-patching
+  "strawberry-graphql[debug-server,opentelemetry]==0.262.5",  # need to pin version because we're monkey-patching
   "pre-commit",
   "arize[AutoEmbeddings, LLM_Evaluation]",
   "llama-index>=0.10.3",
@@ -137,7 +137,7 @@ container = [
   "opentelemetry-instrumentation-sqlalchemy",
   "opentelemetry-instrumentation-grpc",
   "py-grpc-prometheus",
-  "strawberry-graphql[opentelemetry]==0.262.0",  # need to pin version because we're monkey-patching
+  "strawberry-graphql[opentelemetry]==0.262.5",  # need to pin version because we're monkey-patching
   "uvloop; platform_system != 'Windows'",
   "fast-hdbscan>=0.2.0",
   "numba>=0.60.0",  # https://github.com/astral-sh/uv/issues/6281

{arize_phoenix-8.19.1 → arize_phoenix-8.21.0}/src/phoenix/config.py

@@ -7,9 +7,10 @@ from datetime import timedelta
 from enum import Enum
 from importlib.metadata import version
 from pathlib import Path
-from typing import Optional, cast, overload
+from typing import Any, Optional, Union, cast, overload
 from urllib.parse import quote_plus, urlparse
 
+import wrapt
 from email_validator import EmailNotValidError, validate_email
 
 from phoenix.utilities.logging import log_a_list
@@ -144,7 +145,7 @@ ENV_PHOENIX_DISABLE_RATE_LIMIT = "PHOENIX_DISABLE_RATE_LIMIT"
 ENV_PHOENIX_SECRET = "PHOENIX_SECRET"
 ENV_PHOENIX_DEFAULT_ADMIN_INITIAL_PASSWORD = "PHOENIX_DEFAULT_ADMIN_INITIAL_PASSWORD"
 """
-The initial password for the default admin account, which defaults to ‘admin’ if not
+The initial password for the default admin account, which defaults to 'admin' if not
 explicitly set. Note that changing this value will have no effect if the default admin
 record already exists in the database. In such cases, the default admin password must
 be updated manually in the application.
@@ -615,17 +616,138 @@ GENERATED_INFERENCES_NAME_PREFIX = "phoenix_inferences_"
 WORKING_DIR = get_working_dir()
 """The work directory for saving, loading, and exporting data."""
 
-ROOT_DIR = WORKING_DIR
-EXPORT_DIR = ROOT_DIR / "exports"
-INFERENCES_DIR = ROOT_DIR / "inferences"
-TRACE_DATASETS_DIR = ROOT_DIR / "trace_datasets"
 
+class DirectoryError(Exception):
+    def __init__(self, message: Optional[str] = None) -> None:
+        if message is None:
+            message = (
+                "Local storage is not configured. Please set the "
+                "PHOENIX_WORKING_DIR environment variable to fix this."
+            )
+        super().__init__(message)
+
+
+def get_env_postgres_connection_str() -> Optional[str]:
+    pg_user = os.getenv(ENV_PHOENIX_POSTGRES_USER)
+    pg_password = os.getenv(ENV_PHOENIX_POSTGRES_PASSWORD)
+    pg_host = os.getenv(ENV_PHOENIX_POSTGRES_HOST)
+    pg_port = os.getenv(ENV_PHOENIX_POSTGRES_PORT)
+    pg_db = os.getenv(ENV_PHOENIX_POSTGRES_DB)
+
+    if pg_host and ":" in pg_host:
+        pg_host, parsed_port = pg_host.split(":")
+        pg_port = pg_port or parsed_port  # use the explicitly set port if provided
+
+    if pg_host and pg_user and pg_password:
+        encoded_password = quote_plus(pg_password)
+        connection_str = f"postgresql://{pg_user}:{encoded_password}@{pg_host}"
+        if pg_port:
+            connection_str = f"{connection_str}:{pg_port}"
+        if pg_db:
+            connection_str = f"{connection_str}/{pg_db}"
+
+        return connection_str
+    return None
+
+
+def _no_local_storage() -> bool:
+    """
+    Check if we're using a postgres database by checking if postgres connection string is set
+    and a working directory was not explicitly set.
+    """
+    return get_env_postgres_connection_str() is not None and getenv(ENV_PHOENIX_WORKING_DIR) is None
+
+
+class RestrictedPath(wrapt.ObjectProxy):  # type: ignore[misc]
+    """
+    This wraps pathlib.Path and will raise a DirectoryError if no local storage is configured.
+
+    Users can forego configuring a working directory if they are using a postgres database. If this
+    condition is met, the working directory path wrapped by this object will raise an error when
+    accessed in any way.
+    """
+
+    def __init__(self, wrapped: Union[str, Path]) -> None:
+        super().__init__(Path(wrapped))
+        self.__wrapped__: Path
+
+    def _check_forbidden(self) -> None:
+        if _no_local_storage():
+            raise DirectoryError()
+        return
+
+    def __getattr__(self, name: str) -> Any:
+        attr = getattr(self.__wrapped__, name)
+
+        if callable(attr):
+
+            def wrapped_attr(*args: Any, **kwargs: Any) -> Any:
+                result = attr(*args, **kwargs)
+                if isinstance(result, Path):
+                    self._check_forbidden()
+                    return RestrictedPath(result)
+                elif hasattr(result, "__iter__") and not isinstance(result, (str, bytes)):
+                    return (RestrictedPath(p) if isinstance(p, Path) else p for p in result)
+                return result
 
-def ensure_working_dir() -> None:
+            return wrapped_attr
+        else:
+            if isinstance(attr, Path):
+                self._check_forbidden()
+                return RestrictedPath(attr)
+            return attr
+
+    def __str__(self) -> str:
+        self._check_forbidden()
+        return str(self.__wrapped__)
+
+    def __repr__(self) -> str:
+        return f"<RestrictedPath({repr(self.__wrapped__)})>"
+
+    def __fspath__(self) -> str:
+        self._check_forbidden()
+        return str(self.__wrapped__)
+
+    def __truediv__(self, other: Union[str, Path]) -> Path:
+        self._check_forbidden()
+        return self.__wrapped__ / other
+
+    def __itruediv__(self, other: Union[str, Path]) -> Path:
+        self.__wrapped__ /= other
+        self._check_forbidden()
+        return self.__wrapped__
+
+    def __eq__(self, other: object) -> bool:
+        if isinstance(other, RestrictedPath):
+            return bool(self.__wrapped__ == other.__wrapped__)
+        return bool(self.__wrapped__ == other)
+
+    def __hash__(self) -> int:
+        return hash(self.__wrapped__)
+
+    def __len__(self) -> int:
+        return len(self.__wrapped__.parts)
+
+    def __contains__(self, item: str) -> bool:
+        return item in self.__wrapped__.parts
+
+
+ROOT_DIR = RestrictedPath(WORKING_DIR)
+EXPORT_DIR = RestrictedPath(WORKING_DIR / "exports")
+INFERENCES_DIR = RestrictedPath(WORKING_DIR / "inferences")
+TRACE_DATASETS_DIR = RestrictedPath(WORKING_DIR / "trace_datasets")
+
+
+def ensure_working_dir_if_needed() -> None:
     """
     Ensure the working directory exists. This is needed because the working directory
     must exist before certain operations can be performed.
+
+    This is bypassed if a postgres database is configured and a working directory is not set.
     """
+    if _no_local_storage():
+        pass
+
     logger.info(f"📋 Ensuring phoenix working directory: {WORKING_DIR}")
     try:
         for path in (
@@ -644,8 +766,8 @@ def ensure_working_dir() -> None:
         raise
 
 
-# Invoke ensure_working_dir() to ensure the working directory exists
-ensure_working_dir()
+# Invoke ensure_working_dir_if_needed() to ensure the working directory exists
+ensure_working_dir_if_needed()
 
 
 def get_exported_files(directory: Path) -> list[Path]:
@@ -662,6 +784,8 @@ def get_exported_files(directory: Path) -> list[Path]:
     list: list[Path]
         List of paths of the exported files.
     """
+    if _no_local_storage():
+        return []  # Do not attempt to access local storage
     return list(directory.glob("*.parquet"))
 
 
@@ -734,29 +858,6 @@ def get_env_database_connection_str() -> str:
     return f"sqlite:///{working_dir}/phoenix.db"
 
 
-def get_env_postgres_connection_str() -> Optional[str]:
-    pg_user = os.getenv(ENV_PHOENIX_POSTGRES_USER)
-    pg_password = os.getenv(ENV_PHOENIX_POSTGRES_PASSWORD)
-    pg_host = os.getenv(ENV_PHOENIX_POSTGRES_HOST)
-    pg_port = os.getenv(ENV_PHOENIX_POSTGRES_PORT)
-    pg_db = os.getenv(ENV_PHOENIX_POSTGRES_DB)
-
-    if pg_host and ":" in pg_host:
-        pg_host, parsed_port = pg_host.split(":")
-        pg_port = pg_port or parsed_port  # use the explicitly set port if provided
-
-    if pg_host and pg_user and pg_password:
-        encoded_password = quote_plus(pg_password)
-        connection_str = f"postgresql://{pg_user}:{encoded_password}@{pg_host}"
-        if pg_port:
-            connection_str = f"{connection_str}:{pg_port}"
-        if pg_db:
-            connection_str = f"{connection_str}/{pg_db}"
-
-        return connection_str
-    return None
-
-
 def get_env_database_schema() -> Optional[str]:
     if get_env_database_connection_str().startswith("sqlite"):
         return None

{arize_phoenix-8.19.1 → arize_phoenix-8.21.0}/src/phoenix/server/api/types/Project.py

@@ -236,21 +236,6 @@ class Project(Node):
                 stmt = stmt.where(time_range.start <= models.Span.start_time)
             if time_range.end:
                 stmt = stmt.where(models.Span.start_time < time_range.end)
-        if root_spans_only:
-            # A root span is either a span with no parent_id or an orphan span
-            # (a span whose parent_id references a span that doesn't exist in the database)
-            if orphan_span_as_root_span:
-                # Include both types of root spans
-                parent_spans = select(models.Span.span_id).alias("parent_spans")
-                stmt = stmt.where(
-                    ~select(1).where(models.Span.parent_id == parent_spans.c.span_id).exists()
-                    # Note: We avoid using an OR clause with Span.parent_id.is_(None) here
-                    # because it significantly degraded PostgreSQL performance (>10x worse)
-                    # during testing.
-                )
-            else:
-                # Only include explicit root spans (spans with parent_id = NULL)
-                stmt = stmt.where(models.Span.parent_id.is_(None))
         if filter_condition:
             span_filter = SpanFilter(condition=filter_condition)
             stmt = span_filter(stmt)
@@ -274,11 +259,29 @@ class Project(Node):
                 )
             else:
                 stmt = stmt.where(models.Span.id > cursor.rowid)
+        stmt = stmt.order_by(cursor_rowid_column)
+        if root_spans_only:
+            # A root span is either a span with no parent_id or an orphan span
+            # (a span whose parent_id references a span that doesn't exist in the database)
+            if orphan_span_as_root_span:
+                # Include both types of root spans
+                parent_spans = select(models.Span.span_id).alias("parent_spans")
+                candidate_spans = stmt.add_columns(models.Span.parent_id).cte("candidate_spans")
+                stmt = select(candidate_spans).where(
+                    or_(
+                        candidate_spans.c.parent_id.is_(None),
+                        ~select(1)
+                        .where(candidate_spans.c.parent_id == parent_spans.c.span_id)
+                        .exists(),
+                    )
+                )
+            else:
+                # Only include explicit root spans (spans with parent_id = NULL)
+                stmt = stmt.where(models.Span.parent_id.is_(None))
         if first:
             stmt = stmt.limit(
                 first + 1  # overfetch by one to determine whether there's a next page
             )
-        stmt = stmt.order_by(cursor_rowid_column)
         cursors_and_nodes = []
         async with info.context.db() as session:
             span_records = await session.stream(stmt)

arize_phoenix-8.21.0/src/phoenix/server/static/.vite/manifest.json

@@ -0,0 +1,87 @@
+{
+  "_components-BAc4OPED.js": {
+    "file": "assets/components-BAc4OPED.js",
+    "name": "components",
+    "imports": [
+      "_vendor-CEisxXSv.js",
+      "_pages-Dz-gbBPF.js",
+      "_vendor-arizeai-BCTsSnvS.js",
+      "_vendor-codemirror-DIWnRs_7.js",
+      "_vendor-three-C5WAXd5r.js"
+    ]
+  },
+  "_pages-Dz-gbBPF.js": {
+    "file": "assets/pages-Dz-gbBPF.js",
+    "name": "pages",
+    "imports": [
+      "_vendor-CEisxXSv.js",
+      "_vendor-arizeai-BCTsSnvS.js",
+      "_components-BAc4OPED.js",
+      "_vendor-codemirror-DIWnRs_7.js",
+      "_vendor-recharts-Bame54mG.js"
+    ]
+  },
+  "_vendor-CEisxXSv.js": {
+    "file": "assets/vendor-CEisxXSv.js",
+    "name": "vendor",
+    "imports": [
+      "_vendor-three-C5WAXd5r.js"
+    ],
+    "css": [
+      "assets/vendor-Cg6lcjUC.css"
+    ]
+  },
+  "_vendor-Cg6lcjUC.css": {
+    "file": "assets/vendor-Cg6lcjUC.css",
+    "src": "_vendor-Cg6lcjUC.css"
+  },
+  "_vendor-arizeai-BCTsSnvS.js": {
+    "file": "assets/vendor-arizeai-BCTsSnvS.js",
+    "name": "vendor-arizeai",
+    "imports": [
+      "_vendor-CEisxXSv.js"
+    ]
+  },
+  "_vendor-codemirror-DIWnRs_7.js": {
+    "file": "assets/vendor-codemirror-DIWnRs_7.js",
+    "name": "vendor-codemirror",
+    "imports": [
+      "_vendor-CEisxXSv.js",
+      "_vendor-shiki-Cc73E4D-.js"
+    ]
+  },
+  "_vendor-recharts-Bame54mG.js": {
+    "file": "assets/vendor-recharts-Bame54mG.js",
+    "name": "vendor-recharts",
+    "imports": [
+      "_vendor-CEisxXSv.js"
+    ]
+  },
+  "_vendor-shiki-Cc73E4D-.js": {
+    "file": "assets/vendor-shiki-Cc73E4D-.js",
+    "name": "vendor-shiki",
+    "imports": [
+      "_vendor-CEisxXSv.js"
+    ]
+  },
+  "_vendor-three-C5WAXd5r.js": {
+    "file": "assets/vendor-three-C5WAXd5r.js",
+    "name": "vendor-three"
+  },
+  "index.tsx": {
+    "file": "assets/index-Du53xkjY.js",
+    "name": "index",
+    "src": "index.tsx",
+    "isEntry": true,
+    "imports": [
+      "_vendor-CEisxXSv.js",
+      "_vendor-arizeai-BCTsSnvS.js",
+      "_pages-Dz-gbBPF.js",
+      "_components-BAc4OPED.js",
+      "_vendor-three-C5WAXd5r.js",
+      "_vendor-codemirror-DIWnRs_7.js",
+      "_vendor-shiki-Cc73E4D-.js",
+      "_vendor-recharts-Bame54mG.js"
+    ]
+  }
+}