-
Notifications
You must be signed in to change notification settings - Fork 93
Updated RankLLM integration #81
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @clides thanks for the contribution. At the moment, this code isn't tested or documented at a level where it can be merged to langchain-community.
As a rule of thumb the amount of documentation should be similar to the amount of code, and there should be likely some examples with usage.
It looks like there are some breaking changes in this code? Is that correct?
Yep, there are some changes with RankLLM (ie new params, changed function/class names, etc) |
Added more comments as requested |
Updated existing RankLLM integration for langchain-community, with the latest params, function names, etc.
Example with updated usage for RankLLM:
Install all the packages:
Install the document example:
https://github.com/hwchase17/chat-your-data/blob/master/state_of_the_union.txt
Set up the base vector store retriever:
Retrieval without reranking:
All the field arguments to RankLLMRerank:
Retrieval with reranking (default RankLLM model is rank_zephyr):