Skip to content

Back temperature=0 for server as default after #32723#1039

Merged
michalkuligowski merged 3 commits intovllm-project:mainfrom
iboiko-habana:fix32723_main
Feb 26, 2026
Merged

Back temperature=0 for server as default after #32723#1039
michalkuligowski merged 3 commits intovllm-project:mainfrom
iboiko-habana:fix32723_main

Conversation

@iboiko-habana
Copy link
Collaborator

No description provided.

Signed-off-by: Iryna Boiko <iboiko@habana.ai>
Copy link
Collaborator

@kamil-kaczor kamil-kaczor left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request restores the --temperature 0 parameter as the default for vLLM benchmark server commands, reverting a change that was apparently made in issue #32723. The change ensures deterministic and reproducible benchmark results by explicitly setting the temperature sampling parameter.

Changes:

  • Added --temperature 0 flag to the vLLM benchmark serve command in the benchmark template

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@kamil-kaczor kamil-kaczor left a comment

Choose a reason for hiding this comment

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

lgtm

@michalkuligowski michalkuligowski enabled auto-merge (squash) February 26, 2026 09:56
@michalkuligowski michalkuligowski merged commit f5d8681 into vllm-project:main Feb 26, 2026
67 of 68 checks passed
@github-actions
Copy link

✅ CI Passed

All checks passed successfully against the following vllm commit:
dc6de33c3d5e9026cef7b27791dfe0f98e64bbde

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants