Skip to content

UPSTREAM PR #18073: Update README.md incorrect argument#584

Open
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18073-branch_2114L3-patch-2
Open

UPSTREAM PR #18073: Update README.md incorrect argument#584
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18073-branch_2114L3-patch-2

Conversation

@loci-dev
Copy link

Mirrored from ggml-org/llama.cpp#18073

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
https://github.com/ggml-org/llama.cpp/blob/d6a1e18c651a46109cbf2ad3b299581f0651128f/tools/server/README.md?plain=1#L83

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'
@loci-review
Copy link

loci-review bot commented Dec 16, 2025

Explore the complete analysis inside the Version Insights

Performance Analysis Summary - PR #584

Analysis Type: Documentation-only change
Files Modified: 1 (tools/server/README.md)
Code Changes: None

This PR corrects a documentation typo in the llama-server README, changing --n-gpu-layer to --n-gpu-layers. No source code, headers, or binaries were modified. Power consumption analysis confirms 0% change across all binaries. No functions in performance-critical areas were affected.

@loci-dev loci-dev force-pushed the main branch 27 times, most recently from 26e5d36 to 81e654d Compare December 18, 2025 12:15
@loci-dev loci-dev force-pushed the main branch 30 times, most recently from 40ff01f to de0accf Compare December 24, 2025 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants