Skip to content
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

Support DuckDuckGoSearchAPI and TavilySearchAPI as Alternatives to You.com #20

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

songkq
Copy link

@songkq songkq commented Apr 16, 2024

  1. Support DuckDuckGoSearchAPI and TavilySearchAPI as Alternatives to You.com.
  2. When enabling TopicExpert to use DuckDuckGoSearchAPI or TavilySearchAPI, these APIs will return compelete contents instead of snippets as default.
  3. One can setup the search API through editing these environments in secrets.toml:
    Set WEB_SEARCH_API as one of ['DuckDuckGoSearchAPI', 'TavilySearchAPI', 'YouSearchAPI'], using YouSearchAPI as default
    Setup You.com search API key by YDC_API_KEY=<your_youcom_api_key>
    Setup api.tavily.com search API key by TAVILY_API_KEY=<your_api_tavily_com_key>

@shaoyijia
Copy link
Collaborator

Thanks for the effort! We will test this on our side first.

Also, we plan to have a major update to this open-source repo to make it easier to customize or build on top of STORM in the upcoming weeks. We may back up the current branch and directly merge different source support there. Will keep you posted in this PR thread.

@leonstd2015
Copy link

Thanks for your effort!
After running your version, I finnally got the result, but there still have some errors, the openai refuse connection after a while," raise error.APIConnectionError(
openai.error.APIConnectionError: Error communicating with OpenAI: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))"

Have you ever consider change the model to llama3 run by ollama?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants