Skip to content

Commit 925db1d

Browse files
committed
revert statlogger changes
Signed-off-by: alec-flowers <aflowers@nvidia.com>
1 parent d015b13 commit 925db1d

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

vllm/v1/engine/async_llm.py

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ def __init__(
5252
use_cached_outputs: bool = False,
5353
log_requests: bool = True,
5454
start_engine_loop: bool = True,
55-
external_stat_loggers: Optional[list[list[StatLoggerBase]]] = None,
5655
) -> None:
5756
if not envs.VLLM_USE_V1:
5857
raise ValueError(
@@ -76,14 +75,6 @@ def __init__(
7675
loggers.append(
7776
PrometheusStatLogger(vllm_config, engine_index=i))
7877

79-
if external_stat_loggers is not None:
80-
# External stat loggers handle own dp splitting
81-
# Should be in format
82-
# [[stat_logger1-dp1, stat_logger1-dp2],
83-
# [stat_logger2-dp1, stat_logger2-dp2]]
84-
for stat_logger in external_stat_loggers:
85-
loggers.append(stat_logger[i])
86-
8778
self.stat_loggers.append(loggers)
8879

8980
# Tokenizer (+ ensure liveness if running in another process).
@@ -165,7 +156,6 @@ def from_engine_args(
165156
engine_args: AsyncEngineArgs,
166157
start_engine_loop: bool = True,
167158
usage_context: UsageContext = UsageContext.ENGINE_CONTEXT,
168-
external_stat_loggers: Optional[list[list[StatLoggerBase]]] = None,
169159
) -> "AsyncLLM":
170160
"""Create an AsyncLLM from the EngineArgs."""
171161

@@ -181,7 +171,6 @@ def from_engine_args(
181171
log_stats=not engine_args.disable_log_stats,
182172
start_engine_loop=start_engine_loop,
183173
usage_context=usage_context,
184-
external_stat_loggers=external_stat_loggers,
185174
)
186175

187176
def __del__(self):

0 commit comments

Comments
 (0)