-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
feat(community): Port ArxivRetriever to LangChainJS #7250
feat(community): Port ArxivRetriever to LangChainJS #7250
Conversation
Update from main repo
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
@@ -0,0 +1,78 @@ | |||
# ArxivRetriever in LangChain.js (Docs) |
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.
Can you use this template for docs?
@@ -0,0 +1,211 @@ | |||
import axios from 'axios'; // For HTTP requests |
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.
Let's just use fetch
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, sorry for the late response, but our team have just modified the file to use fetch
instead of axios
. We have tested our code and the test cases still passed. Thank you!
1. Removed the import axios line in src/utils/arxiv.ts 2. Removed the dependencies to use axios in "ArxivRetriever in LangChain.js (Docs)" file
Thanks! |
Description:
This PR ports ArxivRetriever from LangChain Python to LangChainJS. An ArxivRetriever component would grant users access to up-to-date research directly from arXiv. This retriever would allow users to pull full text and summaries through natural language queries or arXiv identifiers. Specifically, we have added
Issue:
This PR addresses Port ArxivRetriever to LangChainJS #7095