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

RAG #14

Closed
wants to merge 10 commits into from
Closed

RAG #14

wants to merge 10 commits into from

Conversation

SilasMarvin
Copy link
Owner

This PR adds semantic search as a method for retrieving context.

@SilasMarvin SilasMarvin mentioned this pull request Jun 10, 2024
@PrimeTimeTran
Copy link

PrimeTimeTran commented Jun 10, 2024

@SilasMarvin,
Crawling could suck for legacy projects as they might have huge codebases. One of my previous companies I worked on a project that was almost 15 years old. It was .NET so I had to restart the server every time I made a change. The feedback loop was painful.

I was on a decent system too. Imagine the user is working on ML project where they're also running a local model which requires compute resources. Might increase their startup time a lot too =).

Regardless, I'd like ask/suggest LSP's workspace get configuration method here.

Do you think we could read configs for the project instead of crawling? As a specification, I think we're free to make it work or add additional APIs if we want; right? We might have to do both long term to build a Co pilot Killer so I'm more interested in understanding your thoughts on the LSP spec here =).

@SilasMarvin SilasMarvin marked this pull request as ready for review June 21, 2024 15:32
@SilasMarvin SilasMarvin deleted the silas-rag branch June 25, 2024 04:26
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

2 participants