payi 0.1.0a92__tar.gz → 0.1.0a93__tar.gz

payi-0.1.0a93/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.1.0-alpha.93"
+}

{payi-0.1.0a92 → payi-0.1.0a93}/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.0-alpha.93 (2025-07-01)
+
+Full Changelog: [v0.1.0-alpha.92...v0.1.0-alpha.93](https://github.com/Pay-i/pay-i-python/compare/v0.1.0-alpha.92...v0.1.0-alpha.93)
+
+### Features
+
+* limit connection error log messages ([#333](https://github.com/Pay-i/pay-i-python/issues/333)) ([f454006](https://github.com/Pay-i/pay-i-python/commit/f454006c80beaf09dfe92eba98e4aaeae0e017fd))
+
 ## 0.1.0-alpha.92 (2025-06-30)
 
 Full Changelog: [v0.1.0-alpha.91...v0.1.0-alpha.92](https://github.com/Pay-i/pay-i-python/compare/v0.1.0-alpha.91...v0.1.0-alpha.92)

{payi-0.1.0a92 → payi-0.1.0a93}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: payi
-Version: 0.1.0a92
+Version: 0.1.0a93
 Summary: The official Python library for the payi API
 Project-URL: Homepage, https://github.com/Pay-i/pay-i-python
 Project-URL: Repository, https://github.com/Pay-i/pay-i-python

{payi-0.1.0a92 → payi-0.1.0a93}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "payi"
-version = "0.1.0-alpha.92"
+version = "0.1.0-alpha.93"
 description = "The official Python library for the payi API"
 dynamic = ["readme"]
 license = "Apache-2.0"

{payi-0.1.0a92 → payi-0.1.0a93}/src/payi/_version.py

@@ -1,4 +1,4 @@
 # File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
 __title__ = "payi"
-__version__ = "0.1.0-alpha.92"  # x-release-please-version
+__version__ = "0.1.0-alpha.93"  # x-release-please-version

{payi-0.1.0a92 → payi-0.1.0a93}/src/payi/lib/VertexInstrumentor.py

@@ -88,7 +88,6 @@ class _GoogleVertexRequest(_VertexRequest):
             )
         self._prompt_character_count = 0
         self._candidates_character_count = 0
-        self._model_name: Optional[str] = None
 
     @override
     def process_request(self, instance: Any, extra_headers: 'dict[str, str]', args: Sequence[Any], kwargs: Any) -> bool:
@@ -100,7 +99,7 @@ class _GoogleVertexRequest(_VertexRequest):
             if model and isinstance(model, str):
                 # Extract the model name after the last slash
                 self._model_name = model.split('/')[-1]
-        
+
         if not args:
             return True
         

{payi-0.1.0a92 → payi-0.1.0a93}/src/payi/lib/instrument.py

@@ -1,5 +1,6 @@
 import os
 import json
+import time
 import uuid
 import asyncio
 import inspect
@@ -14,7 +15,7 @@ from dataclasses import dataclass
 import nest_asyncio  # type: ignore
 from wrapt import ObjectProxy  # type: ignore
 
-from payi import Payi, AsyncPayi, __version__ as _payi_version
+from payi import Payi, AsyncPayi, APIConnectionError, __version__ as _payi_version
 from payi.types import IngestUnitsParams
 from payi.lib.helpers import PayiHeaderNames
 from payi.types.ingest_response import IngestResponse
@@ -137,6 +138,7 @@ class PayiInstrumentConfig(TypedDict, total=False):
     proxy: bool
     global_instrumentation: bool
     instrument_inline_data: bool
+    connection_error_logging_window: int
     limit_ids: Optional["list[str]"]
     use_case_name: Optional[str]
     use_case_id: Optional[str]
@@ -222,6 +224,12 @@ class _PayiInstrumentor:
         self._blocked_limits: set[str] = set()
         self._exceeded_limits: set[str] = set()
 
+        self._api_connection_error_last_log_time: float = time.time()
+        self._api_connection_error_count: int = 0
+        self._api_connection_error_window: int = global_config.get("connection_error_logging_window", 60)
+        if self._api_connection_error_window < 0:
+            raise ValueError("connection_error_logging_window must be a non-negative integer")
+
         # default is instrument and ingest metrics
         self._proxy_default: bool = global_config.get("proxy", False)
 
@@ -353,6 +361,9 @@ class _PayiInstrumentor:
             # convert the function call builder to a list of function calls
             ingest_units["provider_response_function_calls"] = list(request._function_call_builder.values())
 
+        if 'resource' not in ingest_units or len(ingest_units['resource']) == 0:
+            ingest_units['resource'] = "system.unknown_model"  # type: ignore[unreachable]
+
         request_json = ingest_units.get('provider_request_json', "")
         if request_json and self._instrument_inline_data is False:
             try:
@@ -402,6 +413,23 @@ class _PayiInstrumentor:
                 if removeBlockedId:
                     self._blocked_limits.discard(limit_id)
 
+    def _process_ingest_connection_error(self, e: APIConnectionError, ingest_units: IngestUnitsParams) -> None:
+        now = time.time()
+
+        if (now - self._api_connection_error_last_log_time) > self._api_connection_error_window:
+            # If previous window had suppressed errors, log the count
+            append = ""
+            if self._api_connection_error_count > 0:
+                append = f", {self._api_connection_error_count} APIConnectionError exceptions in the last {self._api_connection_error_window} seconds"
+
+            # Log the current error
+            self._logger.error(f"Error Pay-i ingesting: connection exception {e}, request {ingest_units}{append}")
+            self._api_connection_error_last_log_time = now
+            self._api_connection_error_count = 0
+        else:
+            # Suppress and count
+            self._api_connection_error_count += 1
+
     async def _aingest_units(self, request: _ProviderRequest) -> Optional[IngestResponse]:
         ingest_response: Optional[IngestResponse] = None
         ingest_units = request._ingest
@@ -437,7 +465,10 @@ class _PayiInstrumentor:
 
             return ingest_response
         except Exception as e:
-            self._logger.error(f"Error Pay-i async ingesting: exception {e}, request {ingest_units}")
+            if isinstance(e, APIConnectionError) and self._api_connection_error_window > 0:
+                self._process_ingest_connection_error(e, ingest_units)
+            else:
+                self._logger.error(f"Error Pay-i async ingesting: exception {e}, request {ingest_units}")
     
         return None         
     
@@ -513,7 +544,10 @@ class _PayiInstrumentor:
                 self._logger.error("No payi instance to ingest units")
 
         except Exception as e:
-            self._logger.error(f"Error Pay-i ingesting: exception {e}, request {ingest_units}")
+            if isinstance(e, APIConnectionError) and self._api_connection_error_window > 0:
+                self._process_ingest_connection_error(e, ingest_units)
+            else:
+                self._logger.error(f"Error Pay-i ingesting: exception {e}, request {ingest_units}")
         
         return None
 

payi-0.1.0a92/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "0.1.0-alpha.92"
-}