Skip to content

fix(docs): correct typos found during code review#20041

Merged
CISC merged 6 commits intoggml-org:masterfrom
marcelpetrick:mpe/typoFixes
Mar 5, 2026
Merged

fix(docs): correct typos found during code review#20041
CISC merged 6 commits intoggml-org:masterfrom
marcelpetrick:mpe/typoFixes

Conversation

@marcelpetrick
Copy link
Contributor

@marcelpetrick marcelpetrick commented Mar 2, 2026

Non-functional changes only:

  • Fixed minor spelling mistakes in comments
  • Corrected typos in user-facing strings
  • No variables, logic, or functional code was modified.

AI was not involved in the creation of this PR. A python-script, which calls codespell, was used to check the whole project. Then a comprehensive manual review of all changes was done, where I rejected ambiguous changes.

Signed-off-by: Marcel Petrick mail@marcelpetrick.it

Non-functional changes only:
- Fixed minor spelling mistakes in comments
- Corrected typos in user-facing strings
- No variables, logic, or functional code was modified.

Signed-off-by: Marcel Petrick <mail@marcelpetrick.it>
@github-actions github-actions bot added documentation Improvements or additions to documentation model Model specific script Script related testing Everything test related Nvidia GPU Issues specific to Nvidia GPUs Vulkan Issues specific to the Vulkan backend examples python python script changes server ggml changes relating to the ggml tensor library for machine learning SYCL https://en.wikipedia.org/wiki/SYCL - GPU programming language Apple Metal https://en.wikipedia.org/wiki/Metal_(API) Ascend NPU issues specific to Ascend NPUs OpenCL Issues specific to the OpenCL backend jinja parser Issues related to the jinja parser labels Mar 2, 2026
Copy link
Collaborator

@taronaeo taronaeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dang, there are a lot of changes. Might take some time to review :)

Co-authored-by: Aaron Teo <taronaeo@gmail.com>
@marcelpetrick
Copy link
Contributor Author

@taronaeo I have tried out that offered action to add your proposed change to the PR. Thank you.

Copy link
Member

@CISC CISC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

supernits :)

marcelpetrick and others added 2 commits March 4, 2026 14:22
Co-authored-by: Sigbjørn Skjæret <sigbjorn.skjaeret@scala.com>
Co-authored-by: Sigbjørn Skjæret <sigbjorn.skjaeret@scala.com>
@marcelpetrick
Copy link
Contributor Author

@CISC Thank you for the input. I have added them as well.

@CISC
Copy link
Member

CISC commented Mar 4, 2026

Merge on green unless anyone objects.

@CISC CISC merged commit 92f7da0 into ggml-org:master Mar 5, 2026
82 of 83 checks passed
bartowski1182 pushed a commit to bartowski1182/llama.cpp that referenced this pull request Mar 10, 2026
* fix(docs): correct typos found during code review

Non-functional changes only:
- Fixed minor spelling mistakes in comments
- Corrected typos in user-facing strings
- No variables, logic, or functional code was modified.

Signed-off-by: Marcel Petrick <mail@marcelpetrick.it>

* Update docs/backend/CANN.md

Co-authored-by: Aaron Teo <taronaeo@gmail.com>

* Revert "Auxiliary commit to revert individual files from 846d1c3"

This reverts commit 02fcf0c7db661d5ff3eff96b2b2db9fdb7213256.

* Update tests/test-backend-ops.cpp

Co-authored-by: Sigbjørn Skjæret <sigbjorn.skjaeret@scala.com>

* Update tests/test-backend-ops.cpp

Co-authored-by: Sigbjørn Skjæret <sigbjorn.skjaeret@scala.com>

---------

Signed-off-by: Marcel Petrick <mail@marcelpetrick.it>
Co-authored-by: Aaron Teo <taronaeo@gmail.com>
Co-authored-by: Sigbjørn Skjæret <sigbjorn.skjaeret@scala.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Apple Metal https://en.wikipedia.org/wiki/Metal_(API) Ascend NPU issues specific to Ascend NPUs documentation Improvements or additions to documentation examples ggml changes relating to the ggml tensor library for machine learning jinja parser Issues related to the jinja parser model Model specific Nvidia GPU Issues specific to Nvidia GPUs OpenCL Issues specific to the OpenCL backend python python script changes script Script related server SYCL https://en.wikipedia.org/wiki/SYCL - GPU programming language testing Everything test related Vulkan Issues specific to the Vulkan backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants