Skip to content

--help may show the wrong default values when used after other arguments #573

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
slaren opened this issue Mar 28, 2023 · 0 comments · Fixed by #697
Closed

--help may show the wrong default values when used after other arguments #573

slaren opened this issue Mar 28, 2023 · 0 comments · Fixed by #697
Labels
bug Something isn't working documentation Improvements or additions to documentation

Comments

@slaren
Copy link
Member

slaren commented Mar 28, 2023

For example, running ./main -b 512 --help will show the help and say that 512 is the default batch size, which is wrong. This may lead to confusion.
@gjmulder gjmulder added bug Something isn't working documentation Improvements or additions to documentation labels Mar 28, 2023
@mvrilo mvrilo mentioned this issue Apr 2, 2023
Merged
Deadsg pushed a commit to Deadsg/llama.cpp that referenced this issue Dec 19, 2023
@abetlen
Merge pull request ggml-org#573 from mzen17/spelling-error-patch
dcc26f7 
Fix typo "lowe-level API" to "low-level API" in the README
@Bearsaerker Bearsaerker mentioned this issue Mar 12, 2025
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix default params for examples/main mvrilo/llama.cpp
2 participants
@slaren @gjmulder