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 RetrieveChat #967

Closed
wants to merge 86 commits into from
Closed

Refactor RetrieveChat #967

wants to merge 86 commits into from

Conversation

thinkall
Copy link
Collaborator

Why are these changes needed?

Refactor RetrieveChat to unify vector db related APIs and make it easier to plug in new vector dbs.

This PR is based off PR #619. We merged PR #619 into the branch "refactor_rag" instead of "main" because @AyushExel doesn't have time to continue working on it, and there are still some issues that need to be resolved in that PR.

Related issue number

Checks

@hsm207
Copy link
Collaborator

hsm207 commented Dec 21, 2023

@thinkall is there a target date to resolve the remaining issues in PR #619 ?

@thinkall
Copy link
Collaborator Author

thinkall commented Dec 22, 2023

@thinkall is there a target date to resolve the remaining issues in PR #619 ?

#619 is closed. There is no target date for this refactor PR as it's now holiday days and I'll be oof in the next month. I suggest you follow the current docs to integrate weaviate db. Sorry for the inconvenience.

@thinkall thinkall closed this Feb 19, 2024
@thinkall
Copy link
Collaborator Author

Check out #1727

@thinkall thinkall deleted the refactor_rag branch March 31, 2024 13:28
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

4 participants