Skip to content

Update README.md incorrect argument#18073

Merged
ngxson merged 1 commit intoggml-org:masterfrom
2114L3:patch-2
Dec 16, 2025
Merged

Update README.md incorrect argument#18073
ngxson merged 1 commit intoggml-org:masterfrom
2114L3:patch-2

Conversation

@2114L3
Copy link
Contributor

@2114L3 2114L3 commented Dec 16, 2025

detected error in readme quoting n-gpu-layer which is incorrect
argument is n-gpu-layers with the 's'
verified correct from to doco on L83

| `-ngl, --gpu-layers, --n-gpu-layers N` | max. number of layers to store in VRAM (default: -1)<br/>(env: LLAMA_ARG_N_GPU_LAYERS) |

Make sure to read the contributing guidelines before submitting a PR

i have read contributing guidelines
documentation typo fix only, no code impacted

typo in Commit f32ca51

n-gpu-layer is incorrect
argument is n-gpu-layers with the 's'
@ngxson

This comment was marked as outdated.

@ngxson ngxson closed this Dec 16, 2025
@ngxson
Copy link
Contributor

ngxson commented Dec 16, 2025

any changes need to be made in arg.cpp instead, which is the source of the docs

@ngxson
Copy link
Contributor

ngxson commented Dec 16, 2025

sorry I misread the change. after verification it's correct that only --n-gpu-layers with 's' is accepted

@ngxson ngxson reopened this Dec 16, 2025
@ngxson ngxson merged commit 5f5f9b4 into ggml-org:master Dec 16, 2025
10 checks passed
Anico2 added a commit to Anico2/llama.cpp that referenced this pull request Jan 15, 2026
n-gpu-layer is incorrect
argument is n-gpu-layers with the 's'
blime4 referenced this pull request in blime4/llama.cpp Feb 5, 2026
n-gpu-layer is incorrect
argument is n-gpu-layers with the 's'
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.

3 participants