-
Notifications
You must be signed in to change notification settings - Fork 50
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
[ERNIEBot Researcher] update langchain #305
Conversation
qingzhong1
commented
Jan 23, 2024
•
edited
Loading
edited
erniebot-agent/applications/erniebot_researcher/tools/semantic_citation_tool.py
Outdated
Show resolved
Hide resolved
erniebot-agent/applications/erniebot_researcher/tools/semantic_citation_tool.py
Outdated
Show resolved
Hide resolved
erniebot-agent/applications/erniebot_researcher/tools/preprocessing.py
Outdated
Show resolved
Hide resolved
service_context = ServiceContext.from_defaults(embed_model=embeddings) | ||
index = load_index_from_storage(storage_context=storage_context, service_context=service_context) | ||
return index | ||
if not abstract and not use_data: |
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.
use_data可以换个名字吗?我还是不是很理解这个字段能够直观表达的意思
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.
use_data是使用相关数据直接建库,不是像full text或者abstract利用path读取数据来建库
erniebot-agent/applications/erniebot_researcher/tools/preprocessing.py
Outdated
Show resolved
Hide resolved
erniebot-agent/applications/erniebot_researcher/tools/preprocessing.py
Outdated
Show resolved
Hide resolved
erniebot-agent/applications/erniebot_researcher/tools/preprocessing.py
Outdated
Show resolved
Hide resolved
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.
LGTM