Skip to content

[LLM runner] Correct the deprecated seq_len CLI help text #22681

Description

@mergennachin

Problem

The llama runner help for --seq_len says to use max_seq_len, but there is no corresponding CLI flag. The implementation falls back from --max_new_tokens to --seq_len, and its runtime log tells users to prefer --max_new_tokens.

This mismatch was found while validating LLM documentation in #22620. The C++ source change is intentionally excluded from that documentation-only PR.

Proposed change

Update the --seq_len help text to recommend --max_new_tokens and clearly distinguish total generation length from new-token count and the model context limit.

Test plan

Build the llama runner and inspect --help; run existing llama runner CI.

cc @larryliu0820 @cccclai @helunwencser @jackzhxng @digantdesai

Activity

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

Metadata

Metadata

Labels

bugmodule: llmIssues related to LLM examples and apps, and to the extensions/llm/ code

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions