continuedev / continuedev/continue
[Best Practices Violation] Do not hard-code an embeddings file batch size
Open
@RomneyDa is already working on this.
Since Feb 5, 2025.
area:indexing
kind:enhancement
needs-triage
- Dominant language
- TypeScript
- Stars
- 36k
- Forks
- 5.4k
- PR merge metrics
- No merged PRs in 30d
Description
Validations
- I believe this is a way to improve. I'll try to join the Continue Discord for questions
- I'm not able to find an open issue that requests the same enhancement
Problem
The above code sets a hard-coded limit on the number of batched files processed by the Continue extension. There is no way to see this number or to override it by the end-user.
This has a secondary impact on what's shown to the end user while embeddings are being calculated. Confusingly, the progress bar seems to be operating off a different number? Or maybe it isn't at all.
That string is actually built in a completely different place:
Whereas the progress bar is built here:
which is then used below to calculate progress percentage:
Solution
- Make it user configurable.
- In the UI, change the phrase "Computing embeddings for X files" such that X is
totalOpsinstead ofLanceDb.results.compute.length, which at that point is actually being called from within a batch
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.