Skip to content

Conversation

@Jackmin801
Copy link
Member

@Jackmin801 Jackmin801 commented Dec 26, 2025

Note

Removes unused prompt logprobs requests to improve vLLM behavior and aligns documentation.

  • In RLConfig.sampling_args.extra_body and token_utils.prepare_sampling_args_for_token_prompts(), remove prompt_logprobs while keeping return_token_ids/return_tokens_as_token_ids and logprobs enabled
  • Update docs/release/TRAJECTORIES.md to no longer recommend prompt_logprobs in sampling_args

Written by Cursor Bugbot for commit b1ee6c9. This will update automatically on new commits. Configure here.

@CLAassistant
Copy link

CLAassistant commented Dec 26, 2025

CLA assistant check
All committers have signed the CLA.

@Jackmin801 Jackmin801 merged commit 214187c into main Dec 27, 2025
6 checks passed
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