-
Notifications
You must be signed in to change notification settings - Fork 440
quote
should be nullable in MessageContentTextAnnotationsFileCitationObject
#263
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
Comments
Jumping in here. We did a fair amount of work in v1 to get this rendering well, and it seems like the quote is no longer returned for any of the v2 endpoints. Some of the older models might still do it, but not the new ones. There are a couple threads on the community server, including this one with response from OAI staff member that it's not supported but "we'll work on" it. Not sure where else to track it, but this OpenAPI repo seems more appropriate than the python API one. But yes, I can also imagine that this causes |
Apologies for the oversight here -- we just rolled out a fix to remove |
Not sure if this is the place to discuss this further, but having the quote available at the client level is going to be crucial for RAG validation. Removing it is a bad move. |
@nknj Is team going to add quotes back? |
Indeed, same issue |
The 'quote' attribute is referenced in the OpenAI documentation: https://platform.openai.com/docs/assistants/deep-dive/managing-threads-and-messages If it has been removed, the documentation needs to be updated. |
In current version, I seem to be experiencing the same issue. See: |
The
quote
property is marked as required, but the server doesn't include them in all the cases.openai-openapi/openapi.yaml
Lines 11362 to 11397 in 893ba52
Example of response that doesn't include it:
The text was updated successfully, but these errors were encountered: