Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[RAG] Update RAG readme #3917

Merged
merged 2 commits into from
Aug 6, 2021
Merged

[RAG] Update RAG readme #3917

merged 2 commits into from
Aug 6, 2021

Conversation

klshuster
Copy link
Contributor

Patch description
Include more instructions for building new indices; mainly, that small datasets should use --indexer-type exact

jianguoz
jianguoz approved these changes Aug 6, 2021
View reviewed changes
Copy link
Contributor

@jianguoz jianguoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much for the added detailed instructions!

@klshuster klshuster merged commit 775dd9e into master Aug 6, 2021
@klshuster klshuster deleted the update_RAG_readme branch August 6, 2021 22:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jianguoz jianguoz jianguoz approved these changes

@mojtaba-komeili mojtaba-komeili Awaiting requested review from mojtaba-komeili

Assignees
No one assigned
Labels
CLA Signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@klshuster @jianguoz @facebook-github-bot