-
Notifications
You must be signed in to change notification settings - Fork 94
Updated search_datasets tool using gemini search + the default #947
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
base: main
Are you sure you want to change the base?
Conversation
Remove outdated features and installation instructions from README and add a link to mkdocs website.
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
|
Ah I just filled the form that was required :) |
|
recheck |
1 similar comment
|
recheck |
|
@joaquinvanschoren can you check if this works now and if it can be merged? If so, I can proceed with the other parts of the project as we discussed |
|
@SubhadityaMukherjee just for my understanding: does the gemini search hinge on using gemini api as backend llm? or can any llm backend be used? |
|
Hey I wanted to initially start with gemini, and if it proved to be useful, then add options for anything else. The perks of using gemini itself was that you wouldnt necessarily run into search limits :) |
Redefined the search_datasts tool while using gemini to also do a "google search" using gemini and not just look at the dataset index when doing a search.
This can be tested using "eclair-client --tool search-datasets --query "tree cover datasets" --use-gemini" after running eclair-server and gemini-cli (following the current documentation)
Also updated the required packages so we can use 'uv' as the venv/package manager.