vLLM + TIS needs logprobs_mode = processed_logprobs? #3463
rawsh
started this conversation in
Ideas / Show and tell
Replies: 1 comment
-
Hmm actually it looks like this config option is not respected at all 😞 Looks like it will be in v0.10.2rc2 (?) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
https://github.com/vllm-project/vllm/blob/4fdd6f5cbf877de7c4de33086fe41bb0ac1d3cf3/docs/usage/v1_guide.md?plain=1#L162
there appears to be a mismatch, trainer logprobs are after temperature scaling while vllm logprobs are before by default.
In 0.10+ there is
logprobs_mode
kwarg that can be set toprocessed_logprobs
. (note: this is only an issue when temperature != 1)Beta Was this translation helpful? Give feedback.
All reactions