Skip to content

Implemented Spellcheck for Llama.cpp #7884

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Ferruolo
Copy link

@Ferruolo Ferruolo commented Jun 11, 2024

  • Self Reported Review Complexity:
    • Review Complexity : Low
    • Review Complexity : Medium
    • Review Complexity : High
  • I have read the contributing guidelines

Implemented basic spellcheck using llama. Fixed binary issues, etc from before.

@mofosyne mofosyne added the Review Complexity : Low Trivial changes to code that most beginner devs (or those who want a break) can tackle. e.g. UI fix label Jun 12, 2024
# Llamacheck: Basic Spellcheck and Grammarcheck using Llama


The attached file provides a basic implementation of LLama to
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The attached file provides a basic implementation of LLama to
The attached file provides a basic implementation of LLama to

We use it as follows:
```console
make llamacheck
./llamacheck <./models/llamacheck.gguf>
Copy link
Collaborator

@HanClinto HanClinto Jul 11, 2024

Choose a reason for hiding this comment

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

End-of-line whitespace is making CI fail the editorconfig check.

Suggested change
./llamacheck <./models/llamacheck.gguf>
./llamacheck <./models/llamacheck.gguf>

llama_backend_free();

return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also needs a final newline at the end of the file.

Suggested change
}
}

@ggerganov ggerganov added the demo Demonstrate some concept or idea, not intended to be merged label Jul 12, 2024
Copy link

@vignesh1507 vignesh1507 left a comment

Choose a reason for hiding this comment

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

Thx for adding spellcheck to the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
demo Demonstrate some concept or idea, not intended to be merged examples Review Complexity : Low Trivial changes to code that most beginner devs (or those who want a break) can tackle. e.g. UI fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants