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

refactor: improve semantic embeddings performance #514

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

deathbeam
Copy link
Collaborator

Previously the semantic embeddings were using older model and not using the query in most optimal way. Now it is using text-embedding-3-small model for reduced cost and better performance, embeddings are stored in 512 dimensions and the embedding process was refactored to be more straightforward and use less API calls by including the query in the initial embedding call.

Also slightly improves the formatting of the embedding prompt to be more readable.

@deathbeam deathbeam added the enhancement New feature or request label Nov 18, 2024
Previously the semantic embeddings were using older model and not using the
query in most optimal way. Now it is using text-embedding-3-small model for
reduced cost and better performance, embeddings are stored in 512
dimensions and the embedding process was refactored to be more
straightforward and use less API calls by including the query in the
initial embedding call.

Also slightly improves the formatting of the embedding prompt to be more
readable.

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
@deathbeam deathbeam merged commit d177720 into CopilotC-Nvim:canary Nov 18, 2024
1 check passed
@deathbeam deathbeam deleted the improve-embeddings branch November 18, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant