Skip to content

Add unit test for dspy.retrievers.Embeddings #8119

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
Open
chenmoneygithub opened this issue Apr 25, 2025 · 3 comments
Open

Add unit test for dspy.retrievers.Embeddings #8119

chenmoneygithub opened this issue Apr 25, 2025 · 3 comments
Assignees
@Krishn1412
Labels
good first issue Good for newcomers

Comments

@chenmoneygithub
Copy link
Collaborator

The code is located at dspy/retrievers/embeddings.py, which is a retriever implemented based on Faiss. We should add unit test for it.

We can use local function to compute the embedding so that the test is able to run on github actions.
@chenmoneygithub chenmoneygithub added the good first issue Good for newcomers label Apr 25, 2025
@Krishn1412
Copy link
Contributor

Hey @chenmoneygithub , can I pick this up?

@chenmoneygithub
Copy link
Collaborator Author

@Krishn1412 assigned to you!

@Krishn1412
Copy link
Contributor

@chenmoneygithub Can you please take a look?PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Krishn1412 Krishn1412

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chenmoneygithub @Krishn1412