Skip to content

Commit ae2cdad

Browse files
kartikxsimon-mo
authored and
dtransposed
committed
Raise error for data-parallel with benchmark_throughput (vllm-project#16737)
Signed-off-by: Kartik Ramesh <kartikx2000@gmail.com> Co-authored-by: Simon Mo <simon.mo@hey.com> Signed-off-by: dtransposed <damian@damian-ml-machine.europe-west3-b.c.jetbrains-grazie.internal>
1 parent 7c2b3b3 commit ae2cdad

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

benchmarks/benchmark_throughput.py

+7
Original file line numberDiff line numberDiff line change
@@ -523,6 +523,13 @@ def validate_args(args):
523523
raise ValueError(
524524
"Tokenizer must be the same as the model for MII backend.")
525525

526+
# --data-parallel is not supported currently.
527+
# https://github.com/vllm-project/vllm/issues/16222
528+
if args.data_parallel_size > 1:
529+
raise ValueError(
530+
"Data parallel is not supported in offline benchmark, \
531+
please use benchmark serving instead")
532+
526533

527534
if __name__ == "__main__":
528535
parser = FlexibleArgumentParser(description="Benchmark the throughput.")

0 commit comments

Comments
 (0)