llama.cpp
`json`: fix additionalProperties, allow space after enum/const
#7840
Merged

`json`: fix additionalProperties, allow space after enum/const #7840

ochafik merged 21 commits into ggml-org:master from ochafik:json-additional
ochafik
json: default additionalProperty to true
12a061c8
json: don't force additional props after normal properties!
8785a6ee
json: allow space after enum/const
87d506f5
github-actions github-actions added testing
github-actions github-actions added examples
github-actions github-actions added python
github-actions github-actions added server
json: update pydantic example to set additionalProperties: false
43f74e0c
HanClinto
ochafik ochafik changed the title `json schemas`: fix additionalProperties, allow space after enum/const `json`: fix additionalProperties, allow space after enum/const 1 year ago
ochafik
ochafik
HanClinto
json: prevent additional props to redefine a typed prop
adca9af2
Merge remote-tracking branch 'origin/master' into json-additional
67434386
port not_strings to python, add trailing space
8b47473d
ochafik ochafik changed the title `json`: fix additionalProperties, allow space after enum/const [WIP] `json`: fix additionalProperties, allow space after enum/const 1 year ago
fix not_strings & port to js+py
4e637560
Update json-schema-to-grammar.cpp
02e26344
fix _not_strings for substring overlaps
0e48ea8e
mofosyne mofosyne added Review Complexity : Medium
Merge remote-tracking branch 'origin/master' into json-additional
322d6113
ochafik
Merge remote-tracking branch 'origin/master' into json-additional
6c859ee4
json: fix additionalProperties default, uncomment tests
2f1a087c
json: add integ. test case for additionalProperties
5c2d3fa1
ochafik ochafik marked this pull request as ready for review 1 year ago
ochafik
HanClinto
HanClinto
json: nit: simplify condition
f714d7f1
HanClinto
HanClinto commented on 2024-06-23
HanClinto
HanClinto
reformat grammar integ tests w/ R"""()""" strings where there's escapes
3c64db18
ochafik
Merge remote-tracking branch 'origin/master' into json-additional
65680f94
HanClinto
ochafik ochafik changed the title [WIP] `json`: fix additionalProperties, allow space after enum/const `json`: fix additionalProperties, allow space after enum/const 1 year ago
HanClinto
Merge remote-tracking branch 'origin/master' into json-additional
35bbac1e
ochafik
HanClinto
ochafik
HanClinto
HanClinto
HanClinto approved these changes on 2024-06-25
Merge remote-tracking branch 'origin/master' into json-additional
c6df6cee
Merge remote-tracking branch 'origin/master' into json-additional
fba3a864
update # tokens in server test: consts can now have trailing space
23beed22
ochafik ochafik merged 6777c544 into master 1 year ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone