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.

Sign up for GitHub

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

Jump to bottom

[skip ci] breaking changes of store and index apis #322

Draft
wants to merge 33 commits into
base: main
Choose a base branch
from
Draft

[skip ci] breaking changes of store and index apis #322

wants to merge 33 commits into from
1,264 −608

Conversation

ouonline
Copy link
Contributor

No description provided.

@ouonline ouonline force-pushed the index-api branch 6 times, most recently from 336f986 to 5d56948 Compare October 25, 2024 01:18
@ouonline ouonline changed the title [skip ci]Index api [skip ci] breaking changes of store and index apis Oct 28, 2024
@ouonline ouonline force-pushed the index-api branch 3 times, most recently from 1c11e33 to 1eae574 Compare October 28, 2024 11:39
wzh1994
wzh1994 reviewed Oct 29, 2024
View reviewed changes
lazyllm/tools/rag/__init__.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/base_index.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/base_store.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/index.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/base_store.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/store.py Outdated Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Show resolved Hide resolved
lazyllm/tools/rag/index.py Outdated Show resolved Hide resolved
@wzh1994
Copy link
Contributor

wzh1994 commented Nov 1, 2024

需要一个完整的RAG例子，使用了

  1. milvus store和default index
  2. 使用了map store和Embedding index（store=milvus）

wzh1994
wzh1994 reviewed Nov 1, 2024
View reviewed changes
lazyllm/tools/rag/doc_node.py Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Show resolved Hide resolved
lazyllm/tools/rag/embed_utils.py Show resolved Hide resolved
lazyllm/tools/rag/chroma_store.py Show resolved Hide resolved
lazyllm/tools/rag/milvus_store.py Show resolved Hide resolved
lazyllm/tools/rag/milvus_store.py Show resolved Hide resolved
lazyllm/tools/rag/store_base.py Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Show resolved Hide resolved
lazyllm/tools/rag/doc_impl.py Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wzh1994 wzh1994 wzh1994 left review comments

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

Successfully merging this pull request may close these issues.
3 participants
@ouonline @wzh1994 @lwj-st