-
Notifications
You must be signed in to change notification settings - Fork 16.1k
server: /v1/responses (partial) #18486
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
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
47134fc
from previous PR
c41a6d7
Make instruction(system) as first message
aa2238e
Convert [input_message] (text/image/file)
fd0a13b
Rename convert_responses_to_chatcmpl(body) -> response_body
f4a87c0
Initial tool call support
6e47dea
Erase instructions field from chatcmpl body
313ea1e
Feed reasoning texts to chat template
7d7058b
Use std::vector instead of opaque json array
e550290
Make output_item.added events consistent
97e649e
Move `server_task_result_cmpl_partial::update` from header to source
d9dca02
Match ID of output_item.added and .done events
cd9b4cf
Add function_call only if there is no "fc_" prefix
6c200df
Add function call output at non-streaming API
63c6013
Test if ID is persistent
f232a1b
Add doc
8a2dd2d
Fix style - use trailing comma
42a6eb3
Rewrite state management
5e1f65c
catch up with upstream/master
951fe42
Fix style - "type" is the first item of SSE data
ebb6438
Explicitly check "instructions" from response_body
cf83e1a
Make lambdas static
0d5e3de
Check if reasoning content exists
5ac23d2
Add `oai_resp_id` to task_result_state(also initialized at ctor), ser…
da3ed76
Reject `input_file` since it is not supported by chatcmpl
96995a6
Add "fc_" prefix to non-straming function call id as coderabbit point…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.