-
Notifications
You must be signed in to change notification settings - Fork 7.2k
Added DuckDuckGo Search Tool and Agent in AutoGen Extensions #6682
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
Conversation
@microsoft-github-policy-service agree |
python/packages/autogen-ext/src/autogen_ext/agents/duckduckgo_search/_duckduckgo_agent.py
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/tools/web_search/_duckduckgo_search.py
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/agents/duckduckgo_search/README.md
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/agents/duckduckgo_search/README.md
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/agents/duckduckgo_search/README.md
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/tools/web_search/_duckduckgo_search.py
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/tools/web_search/_duckduckgo_search.py
Outdated
Show resolved
Hide resolved
python/packages/autogen-ext/src/autogen_ext/tools/web_search/_duckduckgo_search.py
Outdated
Show resolved
Hide resolved
1. Added return_value_as_string function, added corresponding example in API doc 2. Removed some stray inits still present, and fixed imports
undo truncate content in output which was done for testing
python/packages/autogen-ext/src/autogen_ext/tools/web_search/_duckduckgo_search.py
Outdated
Show resolved
Hide resolved
Hi Eric (@ekzhu), really appreciate your help and review on this one! :) Could you please let me know if there's anything else needed before we could merge this? |
@varadsrivastava there are still many issues with the PR. If you are using code generation tool I recommend adding existing unit tests as part of the context to properly generate unit tests. Also, make sure to run type checks and unit tests locally before requesting for review. I am fixing this for you this time. In the future please follow the developer guide. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6682 +/- ##
==========================================
+ Coverage 80.33% 80.42% +0.09%
==========================================
Files 232 234 +2
Lines 17688 17863 +175
==========================================
+ Hits 14209 14367 +158
- Misses 3479 3496 +17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Dear @varadsrivastava, unfortunately we had to remove these changes from our code base because of potential DMCA violation. You are welcome to contribute it as an independent package |
Why are these changes needed?
Adds a specialized AutoGen agent that uses DuckDuckGo for web searches, providing privacy-focused search capabilities without requiring API keys.
Related issue number
Checks