Skip to content

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
Collaborator

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
Collaborator

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
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