Skip to content

Embedder#1

Open
ArnabChatterjee20k wants to merge 10 commits into
masterfrom
embedder
Open

Embedder#1
ArnabChatterjee20k wants to merge 10 commits into
masterfrom
embedder

Conversation

@ArnabChatterjee20k
Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

- Introduced `EmbeddingConfig` struct for managing embedding model configurations.
- Implemented `EmbeddingClient` for handling model initialization and embedding generation.
- Added environment variable support for model selection, cache directory, and pool size.
- Created a main function to demonstrate model loading and embedding generation with sample texts.
- Integrated `fastembed` library for embedding functionalities.
Comment thread .gitignore
@@ -0,0 +1,2 @@
/target
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make sure we are not ignoring the onnx cache here so start the server directly with the onnx cache to have a high startup time

@ArnabChatterjee20k
Copy link
Copy Markdown
Member Author

@copilot create a pr fixing the dockerfile

Copy link
Copy Markdown

Copilot AI commented May 22, 2026

@ArnabChatterjee20k I've opened a new pull request, #2, to work on those changes. Once the pull request is ready, I'll request review from you.

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.

2 participants