-
Notifications
You must be signed in to change notification settings - Fork 400
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
"additionalProperties" should not be a required field #844
Comments
@sgsdxzy, the latest version of Outlines uses the Updating to |
After upgrading to
is still rejected by outlines because neither |
@sgsdxzy Could you please try my branch and report back whether it's working for your use case?
Input script:
Output: |
@lapp0 Thanks, I can confirm your branch works correctly. |
Describe the issue as clearly as possible:
Currently for object,
"additionalProperties"
is required to be a dict. However, it can be optional orfalse
.Steps/code to reproduce the bug:
A simple schema, sent to vLLM/Aphrodite-engine as
guided_json
. The properties and types ofparameters
cannot be known beforehand because the function signature varies for different tools. This schema works fine inlm-format-enforcer
Expected result:
Error message:
Outlines/Python version information:
Version information
Context for the issue:
Affects guided json generation for function calling in vLLM/Aphrodite-engine.
The text was updated successfully, but these errors were encountered: