Skip to content

common : bring back missing args, add env var duplication check - #9375

Merged
ngxson merged 4 commits into
ggml-org:masterfrom
ngxson:xsn/missing-args
Sep 8, 2024
Merged

ngxson merged 4 commits into
ggml-org:masterfrom
ngxson:xsn/missing-args

Conversation

@ngxson

@ngxson ngxson commented Sep 8, 2024 •

Copy link
Copy Markdown
Collaborator

Part of my code in #9308 was auto-generated using a script. The script takes gpt_params_print_usage as reference, so some args having handler code but not in usage message was dropped during the migration.

This PR bring back these args:

  • --prio
  • --prio-batch
  • --prio-draft
  • -Cbd, --cpu-mask-batch-draft (note: this arg didn't had a handler in old version)
  • --prio-batch-draft
  • --perplexity (note: same as --all-logits)
  • --output-file (note: same as -o)
  • --save-all-logits, --kl-divergence-base
  • --chunk (note: same as --from-chunk)

Some args like -i, -if, --in-prefix, was mislabeled as LLAMA_EXAMPLE_INFILL while they should be in LLAMA_EXAMPLE_MAIN

Also add duplication check for env var handlers


@ngxson
ngxson requested review from ggerganov and slaren September 8, 2024 15:27
@github-actions github-actions Bot added testing Everything test related examples labels Sep 8, 2024
@ngxson
ngxson merged commit 3f7ccfd into ggml-org:master Sep 8, 2024
dsx1986 pushed a commit to dsx1986/llama.cpp that referenced this pull request Oct 29, 2024
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 15, 2024
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 18, 2024
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

* correct default values
fukuro-kun pushed a commit to fukuro-kun/fukuro-llama-cpp-turboquant that referenced this pull request Jul 5, 2026
…-org#9375)

* common : bring back missing args

* move duplication check to test-arg-parser

* add check for duplicated env var

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

Labels

examples testing Everything test related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants