-
Notifications
You must be signed in to change notification settings - Fork 241
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
tools: add elasticsearch retriever #115
base: main
Are you sure you want to change the base?
Conversation
@tianjing-li tagging you because this is a new version of the previously closed PR here |
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.
Minor change then good to go, you'll also need to sign all your previous commits.
You can setup a GPG key (see the readme) and then run the following:
git reset $(git merge-base main $(git branch --show-current))
^it undos all your commits and adds it to you local changes
then
git commit -S -m "your message"
git push --force
5c893de
to
03bca33
Compare
Bumping for reviews |
@@ -1,4 +1,4 @@ | |||
import { DEFAULT_CHAT_TOOL } from '@/cohere-client'; | |||
import {DEFAULT_CHAT_TOOL, DEFAULT_ELASTIC_SEARCH_TOOL} from '@/cohere-client'; |
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.
Would you mind removing the FE changes? Since the community features are not required to run the toolkit, this could lead to errors or confusing UX.
Thank you for contributing to the Cohere Toolkit!
PR title: "Add search to ELK cluster"
PR message: Delete this entire checklist and replace with
elasticsearch