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.
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
[Frontend] Chat-based Embeddings API #9759
[Frontend] Chat-based Embeddings API #9759
Changes from 41 commits
1b91750
61e0fcf
c62be47
cc999b1
9ed87c1
efa7c6f
ab9297e
5a4f271
4a969b4
279b9ce
7de803f
c1ef363
a10fa85
89e0710
81b94de
a79d3b2
8b950dd
2c91855
f5e72ff
9cd1ac3
d775150
f2b5846
4a25806
bbcfc6a
b6820b7
fed887a
1774b27
c94aa93
e2ecbcd
fbbd8b1
50ad3aa
9c1df21
8049030
a387845
d80ec7e
ea5fd96
b05ede6
dba9806
557c9ef
8c8ee96
c3ba030
46f316f
1179f66
eb4b235
bf46a16
7f188f9
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Just leaving this as a thought here: should we perhaps have a fork of the openai client that support our extensions explicitly?
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.
This sounds good, but not sure whether we have bandwidth to maintain it 😅
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.
I suggest opening an issue for this.