Implemented Selenium based web browsing. #1397
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Replaced the default web browsing function with one that uses selenium to gather information with a visual and an overlay. Included a small bug fix for the missing google api key that would attempt to use official google with default api keys from the template.
Noticed issues with flake8 formatting and fixed them.
Void-n-Null
changed the title
p-i-
approved these changes
Apr 14, 2023
|
Since this commit, autogpt fails to start up; I think you need to update |
hdkiller
reviewed
Apr 14, 2023
| def create_message(chunk, question): | ||
| return { | ||
| "role": "user", | ||
| "content": f"\"\"\"{chunk}\"\"\" Using the above text, please answer the following question: \"{question}\" -- if the question cannot be answered using the text, please summarize the text." |
There was a problem hiding this comment.
Do we need “please” in the prompt?
5 tasks
bszollosinagy
pushed a commit
to bszollosinagy/Auto-GPT
that referenced
this pull request
Apr 15, 2023
…ing each others efforts in many aspects. * added a new config parameter in .env to control which kind of browser the user wants: headless or full Selenium with Chrome * restored browse_website() to commands.py * PR Significant-Gravitas#1397 introduced a working Selenium adapter, but inadvertently clobbered PR Significant-Gravitas#968, and replicated most of the stuff in browse.py, but based on an old version, without any merge conflicts. This is now rectified by moving Selenium code into browse.py, and reducing duplication as much as possible. * there was a small typo, because an object reference was also returned along with the links in the link scraper. * listed the PROs and CONs of each browser in the source code
Closed
5 tasks
sindlinger
pushed a commit
to Orgsindlinger/Auto-GPT-WebUI
that referenced
this pull request
Sep 25, 2024
* Implemented Selenium based web browing. Replaced the default web browsing function with one that uses selenium to gather information with a visual and an overlay. Included a small bug fix for the missing google api key that would attempt to use official google with default api keys from the template. * Fixed flake8 issues.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Replaced the default web browsing function with one that uses selenium to gather information with a visual and an overlay.
Included a small bug fix for the missing google api key that would attempt to use official google with default api keys from the template.
Background
Changes
Documentation
Test Plan
PR Quality Checklist