Skip to content

Conversation

ekagra-ranjan
Copy link
Contributor

@ekagra-ranjan ekagra-ranjan commented Sep 8, 2025

Update bench doc with mtbench, blazedit, spec bench instructions. Merge after these
#23605
#23563

Signed-off-by: Ekagra Ranjan <3116519+ekagra-ranjan@users.noreply.github.com>
Signed-off-by: Ekagra Ranjan <3116519+ekagra-ranjan@users.noreply.github.com>
@mergify mergify bot added the performance Performance-related issues label Sep 8, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the benchmarking documentation to include instructions for mtbench, blazedit, and spec bench. The changes are mostly good, but I found a duplicated entry for HuggingFace-MTBench in the dataset overview table which should be removed.

Signed-off-by: Ekagra Ranjan <3116519+ekagra-ranjan@users.noreply.github.com>
@ekagra-ranjan ekagra-ranjan changed the title Update bench doc with mtbench, blazedit, spec bench [Benchmark] Update bench doc with mtbench, blazedit, spec bench Sep 8, 2025
@ywang96 ywang96 enabled auto-merge (squash) September 9, 2025 18:05
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 9, 2025
@ywang96 ywang96 merged commit 0dc9cbb into vllm-project:main Sep 9, 2025
25 checks passed
skyloevil pushed a commit to skyloevil/vllm that referenced this pull request Sep 13, 2025
…-project#24450)

Signed-off-by: Ekagra Ranjan <3116519+ekagra-ranjan@users.noreply.github.com>
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
…-project#24450)

Signed-off-by: Ekagra Ranjan <3116519+ekagra-ranjan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance-related issues ready ONLY add when PR is ready to merge/full CI is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants