-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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: integrating a new tool called jina search #14317
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
JINA_SEARCH_URL_ENDPOINT = "https://s.jina.ai/" | ||
|
||
|
||
class JinaToolSpec(BaseToolSpec): |
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.
oh neat, this doesn't require an API key?
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.
Where i mentioned that it requires API key?
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.
You didn't -- I was just surprised that it a service like this works without one
Pull Request Description
This pull request integrates a new tool called
jina
into our platform, addressing the limitation of LLMs regarding their fixed knowledge cutoff.Jina search
provides dynamic grounding capabilities by accessing the latest information from the web, thus enhancing the factuality and relevance of responses.Fixes # (issue not specified)
New Package?
Version Bump?
Type of Change
How Has This Been Tested?
Suggested Checklist:
make format; make lint
to appease the lint gods