openlit 1.34.14__py3-none-any.whl → 1.34.16__py3-none-any.whl

openlit/instrumentation/azure_ai_inference/utils.py

@@ -3,104 +3,185 @@ Azure AI Inference OpenTelemetry instrumentation utility functions
 """
 import time
 
-from opentelemetry.sdk.resources import SERVICE_NAME, TELEMETRY_SDK_NAME, DEPLOYMENT_ENVIRONMENT
 from opentelemetry.trace import Status, StatusCode
 
 from openlit.__helpers import (
     calculate_ttft,
     response_as_dict,
     calculate_tbt,
-    extract_and_format_input,
     get_chat_model_cost,
-    create_metrics_attributes,
-    otel_event,
-    concatenate_all_contents
+    get_embed_model_cost,
+    common_span_attributes,
+    record_completion_metrics,
+    record_embedding_metrics,
 )
 from openlit.semcov import SemanticConvention
 
-def process_chunk(self, chunk):
+def format_content(messages):
+    """
+    Process a list of messages to extract content.
+    """
+
+    formatted_messages = []
+    for message in messages:
+        role = message.get("role", "user")
+        content = message.get("content", "")
+
+        if isinstance(content, list):
+            content_str = ", ".join(
+                f'{item["type"]}: {item["text"] if "text" in item else item.get("image_url", "")}'
+                if "type" in item else f'text: {item.get("text", "")}'
+                for item in content
+            )
+            formatted_messages.append(f'{role}: {content_str}')
+        else:
+            formatted_messages.append(f'{role}: {content}')
+
+    return '\n'.join(formatted_messages)
+
+def process_chunk(scope, chunk):
     """
     Process a chunk of response data and update state.
     """
 
     end_time = time.time()
     # Record the timestamp for the current chunk
-    self._timestamps.append(end_time)
+    scope._timestamps.append(end_time)
 
-    if len(self._timestamps) == 1:
+    if len(scope._timestamps) == 1:
         # Calculate time to first chunk
-        self._ttft = calculate_ttft(self._timestamps, self._start_time)
+        scope._ttft = calculate_ttft(scope._timestamps, scope._start_time)
 
     chunked = response_as_dict(chunk)
 
     # Collect message IDs and aggregated response from events
-    if (len(chunked.get('choices')) > 0 and ('delta' in chunked.get('choices')[0] and
-        'content' in chunked.get('choices')[0].get('delta'))):
+    choices = chunked.get("choices", [])
+    if choices and "delta" in choices[0]:
+        delta = choices[0]["delta"]
+
+        # Handle content
+        content = delta.get("content")
+        if content:
+            scope._llmresponse += content
+
+        # Handle reasoning content (if present)
+        reasoning_content = delta.get("reasoning_content")
+        if reasoning_content:
+            if not hasattr(scope, "_reasoning_content"):
+                scope._reasoning_content = ""
+            scope._reasoning_content += reasoning_content
 
-        if content := chunked.get('choices')[0].get('delta').get('content'):
-            self._llmresponse += content
+        # Handle finish_reason (appears in final chunk)
+        finish_reason = chunked.get("choices")[0].get("finish_reason")
+        if finish_reason:
+            scope._finish_reason = finish_reason
+            scope._end_time = time.time()
 
-    if chunked.get('choices')[0].get('finish_reason') is not None:
-        self._finish_reason = chunked.get('choices')[0].get('finish_reason')
+        # Handle tool calls in streaming - optimized
+        delta_tools = delta.get("tool_calls")
+        if delta_tools:
+            scope._tools = scope._tools or []
 
-    if chunked.get('usage') is not None:
-        self._input_tokens = chunked.get('usage').get('prompt_tokens')
-        self._response_id = chunked.get('id')
-        self._response_model = chunked.get('model')
-        self._output_tokens = chunked.get('usage').get('completion_tokens')
+            for tool in delta_tools:
+                idx = tool.get("index", 0)
+
+                # Extend list if needed
+                scope._tools.extend([{}] * (idx + 1 - len(scope._tools)))
+
+                if tool.get("id"):  # New tool (id exists)
+                    func = tool.get("function", {})
+                    scope._tools[idx] = {
+                        "id": tool["id"],
+                        "function": {"name": func.get("name", ""), "arguments": func.get("arguments", "")},
+                        "type": tool.get("type", "function")
+                    }
+                elif scope._tools[idx] and "function" in tool:  # Append args (id is None)
+                    scope._tools[idx]["function"]["arguments"] += tool["function"].get("arguments", "")
+
+    # Handle usage information (typically only in final chunk)
+    if chunked.get("usage"):
+        scope._input_tokens = chunked.get("usage").get("prompt_tokens", 0)
+        scope._output_tokens = chunked.get("usage").get("completion_tokens", 0)
+        # Handle reasoning tokens if present (optional) - check nested structure
+        completion_details = chunked.get("usage", {}).get("completion_tokens_details", {})
+        if "reasoning_tokens" in completion_details:
+            scope._reasoning_tokens = completion_details.get("reasoning_tokens", 0)
+        elif "reasoning_tokens" in chunked.get("usage", {}):
+            scope._reasoning_tokens = chunked.get("usage").get("reasoning_tokens", 0)
 
 def common_chat_logic(scope, pricing_info, environment, application_name, metrics,
-                        event_provider, capture_message_content, disable_metrics, version, is_stream):
+    capture_message_content, disable_metrics, version, is_stream):
     """
     Process chat request and generate Telemetry
     """
 
-    scope._end_time = time.time()
     if len(scope._timestamps) > 1:
         scope._tbt = calculate_tbt(scope._timestamps)
 
-    formatted_messages = extract_and_format_input(scope._kwargs.get('messages', ''))
-    request_model = scope._kwargs.get('model', 'claude-3-opus-20240229')
+    prompt = format_content(scope._kwargs.get("messages", []))
+    request_model = scope._kwargs.get("model", "gpt-4o")
 
     cost = get_chat_model_cost(request_model, pricing_info, scope._input_tokens, scope._output_tokens)
 
-    # Set Span attributes (OTel Semconv)
-    scope._span.set_attribute(TELEMETRY_SDK_NAME, 'openlit')
-    scope._span.set_attribute(SemanticConvention.GEN_AI_OPERATION, SemanticConvention.GEN_AI_OPERATION_TYPE_CHAT)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_SYSTEM, SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_MODEL, request_model)
-    scope._span.set_attribute(SemanticConvention.SERVER_PORT, scope._server_port)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_MAX_TOKENS, scope._kwargs.get('max_tokens', -1))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_STOP_SEQUENCES, scope._kwargs.get('stop', []))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_TEMPERATURE, scope._kwargs.get('temperature', 1.0))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_TOP_K, scope._kwargs.get('top_k', 1.0))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_TOP_P, scope._kwargs.get('top_p', 1.0))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_FREQUENCY_PENALTY,
-                                                          scope._kwargs.get('frequency_penalty', 0.0))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_PRESENCE_PENALTY,
-                                                          scope._kwargs.get('presence_penalty', 0.0))
-    scope._span.set_attribute(SemanticConvention.GEN_AI_RESPONSE_FINISH_REASON, [scope._finish_reason])
+    # Common Span Attributes
+    common_span_attributes(scope,
+        SemanticConvention.GEN_AI_OPERATION_TYPE_CHAT, SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE,
+        scope._server_address, scope._server_port, request_model, scope._response_model,
+        environment, application_name, is_stream, scope._tbt, scope._ttft, version)
+
+    # Span Attributes for Request parameters
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_FREQUENCY_PENALTY, scope._kwargs.get("frequency_penalty", 0.0))
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_MAX_TOKENS, scope._kwargs.get("max_tokens", -1))
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_PRESENCE_PENALTY, scope._kwargs.get("presence_penalty", 0.0))
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_STOP_SEQUENCES, scope._kwargs.get("stop", []))
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_TEMPERATURE, scope._kwargs.get("temperature", 1.0))
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_TOP_P, scope._kwargs.get("top_p", 1.0))
+
+    # Span Attributes for Response parameters
     scope._span.set_attribute(SemanticConvention.GEN_AI_RESPONSE_ID, scope._response_id)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_RESPONSE_MODEL, scope._response_model)
+    scope._span.set_attribute(SemanticConvention.GEN_AI_RESPONSE_FINISH_REASON, [scope._finish_reason])
+    scope._span.set_attribute(SemanticConvention.GEN_AI_RESPONSE_SERVICE_TIER, scope._response_service_tier)
+    scope._span.set_attribute(SemanticConvention.GEN_AI_RESPONSE_SYSTEM_FINGERPRINT, scope._response_service_tier)
+    scope._span.set_attribute(SemanticConvention.GEN_AI_OUTPUT_TYPE, "text" if isinstance(scope._llmresponse, str) else "json")
+
+    # Span Attributes for Cost and Tokens
     scope._span.set_attribute(SemanticConvention.GEN_AI_USAGE_INPUT_TOKENS, scope._input_tokens)
     scope._span.set_attribute(SemanticConvention.GEN_AI_USAGE_OUTPUT_TOKENS, scope._output_tokens)
-    scope._span.set_attribute(SemanticConvention.SERVER_ADDRESS, scope._server_address)
-
-    scope._span.set_attribute(SemanticConvention.GEN_AI_OUTPUT_TYPE,
-                              'text' if isinstance(scope._llmresponse, str) else 'json')
-
-    scope._span.set_attribute(DEPLOYMENT_ENVIRONMENT, environment)
-    scope._span.set_attribute(SERVICE_NAME, application_name)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_IS_STREAM, is_stream)
     scope._span.set_attribute(SemanticConvention.GEN_AI_CLIENT_TOKEN_USAGE, scope._input_tokens + scope._output_tokens)
     scope._span.set_attribute(SemanticConvention.GEN_AI_USAGE_COST, cost)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_SERVER_TBT, scope._tbt)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_SERVER_TTFT, scope._ttft)
-    scope._span.set_attribute(SemanticConvention.GEN_AI_SDK_VERSION, version)
 
-    # To be removed one the change to log events (from span events) is complete
-    prompt = concatenate_all_contents(formatted_messages)
+    # Span Attributes for Reasoning (if present)
+    if hasattr(scope, "_reasoning_tokens") and scope._reasoning_tokens > 0:
+        scope._span.set_attribute(SemanticConvention.GEN_AI_USAGE_REASONING_TOKENS, scope._reasoning_tokens)
+        # Update total token usage to include reasoning tokens
+        scope._span.set_attribute(SemanticConvention.GEN_AI_CLIENT_TOKEN_USAGE,
+                                scope._input_tokens + scope._output_tokens + scope._reasoning_tokens)
+
+    # Span Attributes for Tools - optimized
+    if scope._tools:
+        tools = scope._tools if isinstance(scope._tools, list) else [scope._tools]
+
+        names, ids, args = zip(*[
+            (t.get("function", {}).get("name", ""),
+             str(t.get("id", "")),
+             str(t.get("function", {}).get("arguments", "")))
+            for t in tools if isinstance(t, dict) and t
+        ]) if tools else ([], [], [])
+
+        scope._span.set_attribute(SemanticConvention.GEN_AI_TOOL_NAME, ", ".join(filter(None, names)))
+        scope._span.set_attribute(SemanticConvention.GEN_AI_TOOL_CALL_ID, ", ".join(filter(None, ids)))
+        scope._span.set_attribute(SemanticConvention.GEN_AI_TOOL_ARGS, ", ".join(filter(None, args)))
+
+    # Span Attributes for Content
     if capture_message_content:
+        scope._span.set_attribute(SemanticConvention.GEN_AI_CONTENT_PROMPT, prompt)
+        scope._span.set_attribute(SemanticConvention.GEN_AI_CONTENT_COMPLETION, scope._llmresponse)
+
+        # Add reasoning content if available
+        if hasattr(scope, "_reasoning_content") and scope._reasoning_content:
+            scope._span.set_attribute(SemanticConvention.GEN_AI_CONTENT_REASONING, scope._reasoning_content)
+
+        # To be removed once the change to span_attributes (from span events) is complete
         scope._span.add_event(
             name=SemanticConvention.GEN_AI_CONTENT_PROMPT_EVENT,
             attributes={
@@ -114,112 +195,143 @@ def common_chat_logic(scope, pricing_info, environment, application_name, metric
             },
         )
 
-    choice_event_body = {
-        'finish_reason': scope._finish_reason,
-        'index': 0,
-        'message': {
-            **({'content': scope._llmresponse} if capture_message_content else {}),
-            'role': 'assistant'
-        }
-    }
-
-    # Emit events
-    for role in ['user', 'system', 'assistant', 'tool']:
-        if formatted_messages.get(role, {}).get('content', ''):
-            event = otel_event(
-                name=getattr(SemanticConvention, f'GEN_AI_{role.upper()}_MESSAGE'),
-                attributes={
-                    SemanticConvention.GEN_AI_SYSTEM: SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE
-                },
-                body = {
-                    # pylint: disable=line-too-long
-                    **({'content': formatted_messages.get(role, {}).get('content', '')} if capture_message_content else {}),
-                    'role': formatted_messages.get(role, {}).get('role', []),
-                    **({
-                        'tool_calls': {
-                            'function': {
-                                # pylint: disable=line-too-long
-                                'name': (scope._tool_calls[0].get('function', {}).get('name', '') if scope._tool_calls else ''),
-                                'arguments': (scope._tool_calls[0].get('function', {}).get('arguments', '') if scope._tool_calls else '')
-                            },
-                            'id': (scope._tool_calls[0].get('id', '') if scope._tool_calls else ''),
-                            'type': 'function'
-                        }
-                    } if role == 'assistant' else {}),
-                    **({
-                        'id': (scope._tool_calls[0].get('id', '') if scope._tool_calls else '')
-                    } if role == 'tool' else {})
-                }
-            )
-            event_provider.emit(event)
-
-    choice_event = otel_event(
-        name=SemanticConvention.GEN_AI_CHOICE,
-        attributes={
-            SemanticConvention.GEN_AI_SYSTEM: SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE
-        },
-        body=choice_event_body
-    )
-    event_provider.emit(choice_event)
-
     scope._span.set_status(Status(StatusCode.OK))
 
+    # Metrics
     if not disable_metrics:
-        metrics_attributes = create_metrics_attributes(
-            service_name=application_name,
-            deployment_environment=environment,
-            operation=SemanticConvention.GEN_AI_OPERATION_TYPE_CHAT,
-            system=SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE,
-            request_model=request_model,
-            server_address=scope._server_address,
-            server_port=scope._server_port,
-            response_model=scope._response_model,
-        )
+        record_completion_metrics(metrics, SemanticConvention.GEN_AI_OPERATION_TYPE_CHAT,
+            SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE, scope._server_address, scope._server_port,
+            request_model, scope._response_model, environment,  application_name, scope._start_time, scope._end_time,
+            scope._input_tokens, scope._output_tokens, cost, scope._tbt, scope._ttft)
 
-        metrics['genai_client_usage_tokens'].record(scope._input_tokens + scope._output_tokens, metrics_attributes)
-        metrics['genai_client_operation_duration'].record(scope._end_time - scope._start_time, metrics_attributes)
-        metrics['genai_server_tbt'].record(scope._tbt, metrics_attributes)
-        metrics['genai_server_ttft'].record(scope._ttft, metrics_attributes)
-        metrics['genai_requests'].add(1, metrics_attributes)
-        metrics['genai_completion_tokens'].add(scope._output_tokens, metrics_attributes)
-        metrics['genai_prompt_tokens'].add(scope._input_tokens, metrics_attributes)
-        metrics['genai_cost'].record(cost, metrics_attributes)
-
-def process_streaming_chat_response(self, pricing_info, environment, application_name, metrics,
-                                    event_provider, capture_message_content=False, disable_metrics=False, version=''):
+def process_streaming_chat_response(scope, pricing_info, environment, application_name, metrics,
+    capture_message_content=False, disable_metrics=False, version=""):
     """
-    Process chat request and generate Telemetry
+    Process streaming chat request and generate Telemetry
     """
 
-    common_chat_logic(self, pricing_info, environment, application_name, metrics,
-                        event_provider, capture_message_content, disable_metrics, version, is_stream=True)
+    common_chat_logic(scope, pricing_info, environment, application_name, metrics,
+        capture_message_content, disable_metrics, version, is_stream=True)
 
 def process_chat_response(response, request_model, pricing_info, server_port, server_address,
-                          environment, application_name, metrics, event_provider, start_time,
-                          span, capture_message_content=False, disable_metrics=False, version='1.0.0', **kwargs):
+    environment, application_name, metrics, start_time, span, capture_message_content=False,
+    disable_metrics=False, version="1.0.0", **kwargs):
     """
     Process chat request and generate Telemetry
     """
 
-    self = type('GenericScope', (), {})()
+    # Create scope object
+    scope = type("GenericScope", (), {})()
+    response_dict = response_as_dict(response)
+
+    scope._start_time = start_time
+    scope._end_time = time.time()
+    scope._span = span
+    scope._llmresponse = " ".join(
+        (choice.get("message", {}).get("content") or "")
+        for choice in response_dict.get("choices", [])
+    )
+    # Handle reasoning content from non-streaming response
+    reasoning_content = response_dict.get("choices", [{}])[0].get("message", {}).get("reasoning_content")
+    if reasoning_content:
+        scope._reasoning_content = reasoning_content
+
+    scope._input_tokens = response_dict.get("usage", {}).get("prompt_tokens", 0)
+    scope._output_tokens = response_dict.get("usage", {}).get("completion_tokens", 0)
+    # Handle reasoning tokens if present (optional) - check nested structure
+    completion_details = response_dict.get("usage", {}).get("completion_tokens_details", {})
+    if "reasoning_tokens" in completion_details:
+        scope._reasoning_tokens = completion_details.get("reasoning_tokens", 0)
+    elif "reasoning_tokens" in response_dict.get("usage", {}):
+        scope._reasoning_tokens = response_dict.get("usage").get("reasoning_tokens", 0)
+    else:
+        scope._reasoning_tokens = 0
+    scope._response_id = response_dict.get("id")
+    scope._response_model = response_dict.get("model")
+    scope._finish_reason = str(response_dict.get("choices", [])[0].get("finish_reason", ""))
+    scope._response_service_tier = str(response_dict.get("system_fingerprint", ""))
+    scope._timestamps = []
+    scope._ttft, scope._tbt = scope._end_time - scope._start_time, 0
+    scope._server_address, scope._server_port = server_address, server_port
+    scope._kwargs = kwargs
+
+    # Handle tool calls
+    if scope._kwargs.get("tools"):
+        scope._tools = response_dict.get("choices", [{}])[0].get("message", {}).get("tool_calls")
+    else:
+        scope._tools = None
+
+    common_chat_logic(scope, pricing_info, environment, application_name, metrics,
+        capture_message_content, disable_metrics, version, is_stream=False)
+
+    return response
+
+def common_embedding_logic(scope, pricing_info, environment, application_name, metrics,
+    capture_message_content, disable_metrics, version):
+    """
+    Process embedding request and generate Telemetry
+    """
+
+    request_model = scope._kwargs.get("model", "text-embedding-3-small")
+
+    cost = get_embed_model_cost(request_model, pricing_info, scope._input_tokens)
+
+    # Common Span Attributes
+    common_span_attributes(scope,
+        SemanticConvention.GEN_AI_OPERATION_TYPE_EMBEDDING, SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE,
+        scope._server_address, scope._server_port, request_model, scope._response_model,
+        environment, application_name, False, 0, scope._end_time - scope._start_time, version)
+
+    # Span Attributes for Request parameters
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_ENCODING_FORMATS, [scope._kwargs.get("encoding_format", "float")])
+    scope._span.set_attribute(SemanticConvention.GEN_AI_REQUEST_USER, scope._kwargs.get("user", ""))
+
+    # Span Attributes for Cost and Tokens
+    scope._span.set_attribute(SemanticConvention.GEN_AI_USAGE_INPUT_TOKENS, scope._input_tokens)
+    scope._span.set_attribute(SemanticConvention.GEN_AI_CLIENT_TOKEN_USAGE, scope._input_tokens)
+    scope._span.set_attribute(SemanticConvention.GEN_AI_USAGE_COST, cost)
+
+    # Span Attributes for Content
+    if capture_message_content:
+        scope._span.set_attribute(SemanticConvention.GEN_AI_CONTENT_PROMPT, str(scope._kwargs.get("input", "")))
+
+        # To be removed once the change to span_attributes (from span events) is complete
+        scope._span.add_event(
+            name=SemanticConvention.GEN_AI_CONTENT_PROMPT_EVENT,
+            attributes={
+                SemanticConvention.GEN_AI_CONTENT_PROMPT: str(scope._kwargs.get("input", "")),
+            },
+        )
+
+    scope._span.set_status(Status(StatusCode.OK))
+
+    # Metrics
+    if not disable_metrics:
+        record_embedding_metrics(metrics, SemanticConvention.GEN_AI_OPERATION_TYPE_EMBEDDING,
+            SemanticConvention.GEN_AI_SYSTEM_AZURE_AI_INFERENCE, scope._server_address, scope._server_port,
+            request_model, scope._response_model, environment, application_name, scope._start_time, scope._end_time,
+            scope._input_tokens, cost)
+
+def process_embedding_response(response, request_model, pricing_info, server_port, server_address,
+    environment, application_name, metrics, start_time, span, capture_message_content=False,
+    disable_metrics=False, version="1.0.0", **kwargs):
+    """
+    Process embedding request and generate Telemetry
+    """
+
+    # Create scope object
+    scope = type("GenericScope", (), {})()
     response_dict = response_as_dict(response)
 
-    # pylint: disable = no-member
-    self._start_time = start_time
-    self._end_time = time.time()
-    self._span = span
-    self._llmresponse = response_dict.get('choices', {})[0].get('message', '').get('content', '')
-    self._input_tokens = response_dict.get('usage').get('prompt_tokens')
-    self._output_tokens = response_dict.get('usage').get('completion_tokens')
-    self._response_model = response_dict.get('model', '')
-    self._finish_reason = response_dict.get('choices', {})[0].get('finish_reason', '')
-    self._response_id = response_dict.get('id', '')
-    self._timestamps = []
-    self._ttft, self._tbt = self._end_time - self._start_time, 0
-    self._server_address, self._server_port = server_address, server_port
-    self._kwargs = kwargs
-
-    common_chat_logic(self, pricing_info, environment, application_name, metrics,
-                        event_provider, capture_message_content, disable_metrics, version, is_stream=False)
+    scope._start_time = start_time
+    scope._end_time = time.time()
+    scope._span = span
+    scope._input_tokens = response_dict.get("usage", {}).get("prompt_tokens", 0)
+    scope._response_model = response_dict.get("model")
+    scope._server_address, scope._server_port = server_address, server_port
+    scope._kwargs = kwargs
+
+    common_embedding_logic(scope, pricing_info, environment, application_name, metrics,
+        capture_message_content, disable_metrics, version)
 
     return response

openlit/instrumentation/litellm/__init__.py

@@ -1,4 +1,3 @@
-# pylint: disable=useless-return, bad-staticmethod-argument, disable=duplicate-code
 """Initializer of Auto Instrumentation of LiteLLM Functions"""
 
 from typing import Collection
@@ -17,15 +16,15 @@ _instruments = ("litellm >= 1.52.6",)
 
 class LiteLLMInstrumentor(BaseInstrumentor):
     """
-    An instrumentor for LiteLLM's client library.
+    An instrumentor for LiteLLM client library.
     """
 
     def instrumentation_dependencies(self) -> Collection[str]:
         return _instruments
 
     def _instrument(self, **kwargs):
-        application_name = kwargs.get("application_name", "default_application")
-        environment = kwargs.get("environment", "default_environment")
+        application_name = kwargs.get("application_name", "default")
+        environment = kwargs.get("environment", "default")
         tracer = kwargs.get("tracer")
         metrics = kwargs.get("metrics_dict")
         pricing_info = kwargs.get("pricing_info", {})
@@ -33,7 +32,7 @@ class LiteLLMInstrumentor(BaseInstrumentor):
         disable_metrics = kwargs.get("disable_metrics")
         version = importlib.metadata.version("litellm")
 
-        # completion
+        # Chat completions
         wrap_function_wrapper(
             "litellm",
             "completion",
@@ -41,6 +40,7 @@ class LiteLLMInstrumentor(BaseInstrumentor):
                   tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
+        # Async chat completions
         wrap_function_wrapper(
             "litellm",
             "acompletion",
@@ -48,6 +48,7 @@ class LiteLLMInstrumentor(BaseInstrumentor):
                   tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
+        # Embeddings
         wrap_function_wrapper(
             "litellm",
             "embedding",
@@ -55,6 +56,7 @@ class LiteLLMInstrumentor(BaseInstrumentor):
                   tracer, pricing_info, capture_message_content, metrics, disable_metrics),
         )
 
+        # Async embeddings
         wrap_function_wrapper(
             "litellm",
             "aembedding",
@@ -63,5 +65,4 @@ class LiteLLMInstrumentor(BaseInstrumentor):
         )
 
     def _uninstrument(self, **kwargs):
-        # Proper uninstrumentation logic to revert patched methods
         pass