From 4d15b08b2cd93d440f678bc2fccfda13b36d06de Mon Sep 17 00:00:00 2001 From: Eric Ly <111820150+lyeric2022@users.noreply.github.com> Date: Wed, 2 Jul 2025 16:29:49 -0700 Subject: [PATCH] fix: correct negative step execution time metrics (#3117) Co-authored-by: Eric Ly --- letta/agents/letta_agent.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/letta/agents/letta_agent.py b/letta/agents/letta_agent.py index 750a42b1..c48251b5 100644 --- a/letta/agents/letta_agent.py +++ b/letta/agents/letta_agent.py @@ -464,7 +464,7 @@ class LettaAgent(BaseAgent): ), ) - MetricRegistry().step_execution_time_ms_histogram.record(step_start - get_utc_timestamp_ns(), get_ctx_attributes()) + MetricRegistry().step_execution_time_ms_histogram.record(get_utc_timestamp_ns() - step_start, get_ctx_attributes()) if not should_continue: break @@ -701,7 +701,7 @@ class LettaAgent(BaseAgent): yield f"data: {tool_return.model_dump_json()}\n\n" # TODO (cliandy): consolidate and expand with trace - MetricRegistry().step_execution_time_ms_histogram.record(step_start - get_utc_timestamp_ns(), get_ctx_attributes()) + MetricRegistry().step_execution_time_ms_histogram.record(get_utc_timestamp_ns() - step_start, get_ctx_attributes()) if not should_continue: break