-
Notifications
You must be signed in to change notification settings - Fork 736
Implement raptor indexing engine (collapsed tree) #924
base: main
Are you sure you want to change the base?
Implement raptor indexing engine (collapsed tree) #924
Conversation
@MarouaneMaatouk can you run |
), | ||
] | ||
|
||
summary.append(self.llm.chat(messages).message.content) |
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.
Ideally we could use a TreeSummarize
module here to recursively summarize -- maybe I'll try to add this
|
||
def run(self, *args: Any, **kwargs: Any) -> Any: | ||
"""Run the pipeline.""" | ||
return self.indexing_engine.insert_abstractions(*args, **kwargs) |
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.
Did I miss where querying is implemented? Or is that not done yet?
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.
Not yet, I'll complete this PR sometime this week 👍
8096a98
to
192308f
Compare
hey @MarouaneMaatouk - quick bump on this. once it's ready we'll help you port it over to |
Description [WIP]
Implement RAPTOR: Recursive Abstractive Processing For Tree-Organized Retrieval
paper: https://arxiv.org/pdf/2401.18059.pdf
Fixes # (issue)
Type of Change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Suggested Checklist:
make format; make lint
to appease the lint gods