Skip to content

Updates json gbnf to match llama.cpp example#1098

Merged
martindevans merged 1 commit intoSciSharp:masterfrom
phil-scott-78:new-json-gbnf
Feb 13, 2025
Merged

Updates json gbnf to match llama.cpp example#1098
martindevans merged 1 commit intoSciSharp:masterfrom
phil-scott-78:new-json-gbnf

Conversation

@phil-scott-78
Copy link
Copy Markdown
Contributor

Example had a tendency to start just outputting whitespace until it ran out of tokens without a few of the constraints they added around whitespace generation. This gets it in line with example at https://github.com/ggerganov/llama.cpp/blob/master/grammars/json.gbnf

Example had a tendency to start just outputting whitespace until it ran out of tokens without a few of the constraints they added around whitespace generation
@martindevans
Copy link
Copy Markdown
Member

Thanks for fixing this!

@martindevans martindevans merged commit 92579ed into SciSharp:master Feb 13, 2025
6 checks passed
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