Skip to content

Add --no-min-tokens flag to skip min_tokens in requests#25

Merged
JimmyWhitaker merged 1 commit intomainfrom
add-no-min-tokens-flag
Feb 13, 2026
Merged

Add --no-min-tokens flag to skip min_tokens in requests#25
JimmyWhitaker merged 1 commit intomainfrom
add-no-min-tokens-flag

Conversation

@dsingal0
Copy link
Copy Markdown

@dsingal0 dsingal0 commented Feb 6, 2026

This flag is useful for backends that don't support min_tokens, such as vLLM with speculative decoding enabled.

Changes:

  • Add --no-min-tokens CLI flag in sampling_options
  • Pass no_min_tokens parameter through to TextSampler
  • Skip setting min_tokens when no_min_tokens=True

Usage:
genai-bench benchmark --no-min-tokens ...

Description

Related Issue

Changes

Correctness Tests

Checklist

  • I have rebased my branch on top of the latest main branch (git pull origin main)
  • I have run make check to ensure code is properly formatted and passes all lint checks
  • I have run make test or make test_changed to verify test coverage (~90% required)
  • I have added tests that fail without my code changes (for bug fixes)
  • I have added tests covering variants of new features (for new features)

Additional Information

Add any other context, screenshots, or information about the PR here.

This flag is useful for backends that don't support min_tokens,
such as vLLM with speculative decoding enabled.

Changes:
- Add --no-min-tokens CLI flag in sampling_options
- Pass no_min_tokens parameter through to TextSampler
- Skip setting min_tokens when no_min_tokens=True

Usage:
  genai-bench benchmark --no-min-tokens ...
@JimmyWhitaker JimmyWhitaker merged commit f2c3dd9 into main Feb 13, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants