Skip to content

[Frontend] Support setting logprobs to -1#25031

Merged
DarkLight1337 merged 3 commits intovllm-project:mainfrom
chaunceyjiang:logprobs
Sep 18, 2025
Merged

[Frontend] Support setting logprobs to -1#25031
DarkLight1337 merged 3 commits intovllm-project:mainfrom
chaunceyjiang:logprobs

Conversation

@chaunceyjiang
Copy link
Collaborator

@chaunceyjiang chaunceyjiang commented Sep 17, 2025

Purpose

Follow up #21792
FIX #24956 (comment)

Test Plan

see e2e

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

@mergify mergify bot added the frontend label Sep 17, 2025
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
@chaunceyjiang
Copy link
Collaborator Author

/cc @DarkLight1337 @charlotte12l PTAL.

@DarkLight1337 DarkLight1337 enabled auto-merge (squash) September 18, 2025 08:58
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 18, 2025
@DarkLight1337 DarkLight1337 merged commit cc935fd into vllm-project:main Sep 18, 2025
53 of 54 checks passed
@chaunceyjiang chaunceyjiang deleted the logprobs branch September 19, 2025 01:13
debroy-rh pushed a commit to debroy-rh/vllm that referenced this pull request Sep 19, 2025
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
charlifu pushed a commit to ROCm/vllm that referenced this pull request Sep 25, 2025
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
Signed-off-by: charlifu <charlifu@amd.com>
choprahetarth pushed a commit to Tandemn-Labs/vllm that referenced this pull request Oct 11, 2025
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend 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