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.

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

test: e2e /reranker endpoint #2211

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

dave-gray101
Copy link
Collaborator

@dave-gray101 dave-gray101 commented May 1, 2024

Description

Current Status:
tests-aio-container task cannot use extras image type for size reasons
Best fix is probably to address this after pixi/uv changes #2215 so that a test can be created that only builds rerankers, if size remains a problem


In order to allow rerankers api to be ported to the new service wiring pattern, add a test first to verify all is well

  • Creates a simple e2e test for the /reranker api

  • ran go mod tidy 🗡️

Testing this independently from the service wiring PR - this PR should be merged first after existing code is validated as working with it.

Signed-off-by: Dave Lee <dave@gray101.com>
Copy link

netlify bot commented May 1, 2024

Deploy Preview for localai canceled.

Name Link
🔨 Latest commit 5558e57
🔍 Latest deploy log https://app.netlify.com/sites/localai/deploys/66339a95b3a95500088969de

@dave-gray101 dave-gray101 enabled auto-merge (squash) May 1, 2024 15:51
Signed-off-by: Dave Lee <dave@gray101.com>
Signed-off-by: Dave Lee <dave@gray101.com>
Signed-off-by: Dave Lee <dave@gray101.com>
Signed-off-by: Dave Lee <dave@gray101.com>
Signed-off-by: Dave Lee <dave@gray101.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant