Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Define embedding/ranking/llm request/response format #289

Draft
wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

Define embedding/ranking/llm request/response format #289

wants to merge 26 commits into from

Conversation

lkk12014402
Copy link
Collaborator

@lkk12014402 lkk12014402 commented Jul 10, 2024 •
edited
Loading

Description

define embedding/ranking/llm request/response by referring to OpenAI format

related pr #275

lkk12014402 and others added 17 commits July 10, 2024 23:20
@lkk12014402
define embedding/ranking/llm request/response by referring to openai …
2a5f0f1 
…format.
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
7e49cf4 
for more information, see https://pre-commit.ci
@lkk12014402
update embedding request/response.
e3ce9b9
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
8c46fa9 
for more information, see https://pre-commit.ci
@lkk12014402
update chat completion/embedding request/response format.
7e369c3
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3c1ff7c 
for more information, see https://pre-commit.ci
@lkk12014402
format retrival request/response and output retrieval metadata.
db3acff
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
6d9ed47 
for more information, see https://pre-commit.ci
@lkk12014402
format retrival request/response and output retrieval metadata.
79258e7
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
9b5c798 
for more information, see https://pre-commit.ci
@lkk12014402
format reranking request/response and output rerank metadata.
00ac00a
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3030318 
for more information, see https://pre-commit.ci
@lkk12014402
format llm microservice with openai format.
671efa5
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
60dee7b 
for more information, see https://pre-commit.ci
@lkk12014402
add openai dependency.
0440948
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
bf5650c 
for more information, see https://pre-commit.ci
@lkk12014402
Update api_protocol.py
9a35261
@codecov Codecov
Copy link

codecov bot commented Jul 15, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 99.18033% with 1 line in your changes missing coverage. Please review.

Files Coverage Δ
comps/cores/proto/api_protocol.py 93.10% <100.00%> (+3.21%) ⬆️
comps/utils.py 66.66% <66.66%> (ø)

... and 3 files with indirect coverage changes

@lkk12014402 lkk12014402 mentioned this pull request Aug 5, 2024
Merged
@eero-t eero-t mentioned this pull request Dec 18, 2024
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lvliang-intel lvliang-intel Awaiting requested review from lvliang-intel lvliang-intel will be requested when the pull request is marked ready for review lvliang-intel is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lkk12014402