Skip to content

Commit

Permalink
Remove unnecessary @override decorator and add # noqa: ARG002 com…
Browse files Browse the repository at this point in the history
…ments for unused arguments
  • Loading branch information
ogabrielluiz committed Oct 25, 2024
1 parent 8f40330 commit ee72767
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/backend/base/langflow/services/tracing/langsmith.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
from typing import TYPE_CHECKING, Any

from loguru import logger
from typing_extensions import override

from langflow.schema.data import Data
from langflow.services.tracing.base import BaseTracer
Expand Down Expand Up @@ -63,15 +62,14 @@ def setup_langsmith(self) -> bool:
os.environ["LANGCHAIN_TRACING_V2"] = "true"
return True

@override
def add_trace(
self,
trace_id: str,
trace_id: str, # noqa: ARG002
trace_name: str,
trace_type: str,
inputs: dict[str, Any],
metadata: dict[str, Any] | None = None,
vertex: Vertex | None = None,
vertex: Vertex | None = None, # noqa: ARG002
) -> None:
if not self._ready or not self._run_tree:
return
Expand Down Expand Up @@ -117,10 +115,9 @@ def _convert_to_langchain_type(self, value):
value = str(value)
return value

@override
def end_trace(
self,
trace_id: str,
trace_id: str, # noqa: ARG002
trace_name: str,
outputs: dict[str, Any] | None = None,
error: Exception | None = None,
Expand Down

0 comments on commit ee72767

Please sign in to comment.