⚡️ Speed up method OpikTracer._get_config by 11% in PR #7183 (feat/global_vars_tracing)

To optimize the code for better performance, we can make minimal yet effective changes. These changes mainly focus on.
1. Reducing repetitive calls,
2. Using more efficient default values,
3. Simplifying logic where possible.

Here’s the optimized version of the code.



Here's what we modified to optimize the code.
1. Avoid redundant `None` default parameter checks with `if host or api_key:` and `self.global_vars or {}` directly in instantiation.
2. Unified the environmental variable fetch with no redundant `None` assignment since `os.getenv` returns `None` by default if the variable is not set.
3. Simplified the boolean conversion with a direct `bool(config and self._setup_opik(config, trace_id))`.

These changes are oriented toward reducing function execution time by minimizing branching logic and redundant checks. The core functionality and output remain unchanged.

Author: codeflash-ai[bot]

src/backend/base/langflow/services/tracing/opik.py

@@ -3,6 +3,7 @@
 import os
 import types
 from typing import TYPE_CHECKING, Any
+from uuid import UUID
 
 from langchain_core.documents import Document
 from langchain_core.messages import BaseMessage, HumanMessage, SystemMessage
@@ -60,7 +61,7 @@ def __init__(
         self.global_vars = global_vars or {}
 
         config = self._get_config(self.global_vars)
-        self._ready: bool = self._setup_opik(config, trace_id) if config else False
+        self._ready: bool = bool(config and self._setup_opik(config, trace_id))
         self._distributed_headers = None
 
     @property
@@ -235,14 +236,14 @@ def _convert_to_opik_type(self, value):
 
     @staticmethod
     def _get_config(global_vars) -> dict:
-        if global_vars:
-            host = global_vars.get("OPIK_URL_OVERRIDE", None)
-            api_key = global_vars.get("OPIK_API_KEY", None)
-            workspace = global_vars.get("OPIK_WORKSPACE", None)
+        if global_vars is not None:
+            host = global_vars.get("OPIK_URL_OVERRIDE")
+            api_key = global_vars.get("OPIK_API_KEY")
+            workspace = global_vars.get("OPIK_WORKSPACE")
         else:
-            host = os.getenv("OPIK_URL_OVERRIDE", None)
-            api_key = os.getenv("OPIK_API_KEY", None)
-            workspace = os.getenv("OPIK_WORKSPACE", None)
+            host = os.getenv("OPIK_URL_OVERRIDE")
+            api_key = os.getenv("OPIK_API_KEY")
+            workspace = os.getenv("OPIK_WORKSPACE")
 
         # API Key is mandatory for Opik Cloud and URL is mandatory for Open-Source Opik Server
         if host or api_key: