-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[AI Connector] Change completion subAction schema to be OpenAI compatible #200249
Merged
YulNaumenko
merged 28 commits into
elastic:main
from
YulNaumenko:ai-connector-inference-completion-openai
Dec 18, 2024
Merged
Changes from 9 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
f04ab4c
[AI Connector] Change completion subAction schema to be OpenAI compat…
YulNaumenko af94594
-
YulNaumenko 697c73c
Merge branch 'main' into ai-connector-inference-completion-openai
YulNaumenko 7428e6f
Merge branch 'main' into ai-connector-inference-completion-openai
YulNaumenko f17c678
Merge branch 'ai-connector-inference-completion-openai' of github.com…
YulNaumenko 2847a11
added unified completion support
YulNaumenko 3a03150
-
YulNaumenko f14b3ec
Merge branch 'main' into ai-connector-inference-completion-openai
YulNaumenko bf34578
added dashboard and async iterator
YulNaumenko f3b6a75
fixed headers
YulNaumenko 72041b5
fixed params
YulNaumenko 9cec4cb
Merge branch 'main' into ai-connector-inference-completion-openai
YulNaumenko 79c5c3b
made the regular stream and non-stream working
YulNaumenko 27512b5
Merge branch 'main' into ai-connector-inference-completion-openai
YulNaumenko 8ce2707
Merge branch 'ai-connector-inference-completion-openai' of github.com…
YulNaumenko cc7993b
merge fix
YulNaumenko 166a22b
Merge remote-tracking branch 'upstream/main' into ai-connector-infere…
YulNaumenko e0ee923
tool calls fix
YulNaumenko 042e813
improved
YulNaumenko 6ded610
-
YulNaumenko 36a6f67
streaming
YulNaumenko a70051b
fixed test
YulNaumenko f811503
fixed streaming
YulNaumenko 2d2ebde
Merge remote-tracking branch 'upstream/main' into ai-connector-infere…
YulNaumenko 232fbe9
fixed due to comments
YulNaumenko ed6107b
excluded n
YulNaumenko 4ccb8f3
fixed tests
YulNaumenko 699b0a9
Merge branch 'main' into ai-connector-inference-completion-openai
YulNaumenko 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 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 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 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 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 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 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 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 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.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Am I supposed to do a transform on the OpenAI
function_call
to make it intotool_choice
?kibana/x-pack/plugins/stack_connectors/common/openai/schema.ts
Lines 92 to 103 in b4ccb0c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct, we should stop using the deprecated way of
function_call
usage and migrate totool_choice
andtools