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

Fix the version of duckduckgo-search to avoid empty result issue #4368

Conversation

kinance
Copy link
Contributor

@kinance kinance commented May 23, 2023

Background

Default google search command is returning nothing

Changes

Putting this quick fix for the issue due to the underlying duckduckgo package. Fixing it to 3.0.2 is a workaround for time being. The workaround was proposed in #4350. Resolves #4350

Test Plan

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • [] I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.
  • [] I have run black . and isort . against my code to ensure it passes our linter.

@vercel
Copy link

vercel bot commented May 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) May 23, 2023 2:42pm

@kinance kinance requested a review from ntindle May 23, 2023 14:45
@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.14 🎉

Comparison is base (9c7f567) 64.76% compared to head (e23beab) 64.90%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4368      +/-   ##
==========================================
+ Coverage   64.76%   64.90%   +0.14%     
==========================================
  Files          75       75              
  Lines        3542     3542              
  Branches      520      520              
==========================================
+ Hits         2294     2299       +5     
+ Misses       1078     1071       -7     
- Partials      170      172       +2     
Impacted Files Coverage Δ
autogpt/app.py 43.15% <100.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@k-boikov k-boikov merged commit d4d50fd into Significant-Gravitas:master May 23, 2023
@kinance kinance deleted the workaround-google-search-return-nothing branch May 23, 2023 15:35
kuwork added a commit to kuwork/Auto-GPT that referenced this pull request May 24, 2023
* master:
  Migrate google search to use DDGS.text function (Significant-Gravitas#4383)
  empty commit to see cassette status (Significant-Gravitas#4382)
  Fix the version of duckduckgo-search to avoid empty result issue (Significant-Gravitas#4368)
  Update get_hyperlinks description (Significant-Gravitas#4122)
  CI: set base repo before `gh pr checkout` (Significant-Gravitas#4363)
  Cassettes should be added in the same pull request, without extra pull request (Significant-Gravitas#4355)
  Improve Azure setup wording in docs (Significant-Gravitas#4325)
  Pass command line args as list (Significant-Gravitas#1486)
@ntindle ntindle added this to the v0.4.0 Release milestone May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

SYSTEM: Command google returned: Error: 'NoneType' object is not iterable
3 participants