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

fixes #1821 by installing required drivers and adding options to chro… #1857

Merged
merged 5 commits into from
Apr 16, 2023

Conversation

psvensson
Copy link
Contributor

@psvensson psvensson commented Apr 16, 2023

…medriver

Background

The web_browse functionality has been broken in the Docker container (and possibly outside it) for a day or two.
This PR adds a number of needed libraries and options when invoking the chromium webdriver which fixes the problems (see #1821 for example)

Changes

New libraries apt-got in Dockerfile + more options to the chromium creation

Documentation

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

shahdevansh
shahdevansh previously approved these changes Apr 16, 2023
Copy link

@shahdevansh shahdevansh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me

@bvoo
Copy link
Contributor

bvoo commented Apr 16, 2023

tested and can also confirm working

shahdevansh
shahdevansh previously approved these changes Apr 16, 2023
DarkShadowG
DarkShadowG previously approved these changes Apr 16, 2023
@psvensson psvensson dismissed stale reviews from DarkShadowG and shahdevansh via 97a9867 April 16, 2023 16:00
@nponeccop nponeccop added the B7 label Apr 16, 2023
@nponeccop
Copy link
Contributor

@psvensson There are lint errors now

@psvensson
Copy link
Contributor Author

hmm OK. The only thing left in my PR is one line in the Dockerfile. Is this where the lint errors are?

@psvensson
Copy link
Contributor Author

Ah hang on, I see it. thx

@p-i- p-i- merged commit 5634eee into Significant-Gravitas:master Apr 16, 2023
@nponeccop nponeccop mentioned this pull request Apr 17, 2023
1 task
@vmajor
Copy link

vmajor commented Apr 20, 2023

Does not work for me: Command browse_website returned: Error: Message: Service /home/*****/.wdm/drivers/chromedriver/linux64/112.0.5615.49/chromedriver unexpectedly exited. Status code was: 127

@psvensson
Copy link
Contributor Author

Does not work for me: Command browse_website returned: Error: Message: Service /home/*****/.wdm/drivers/chromedriver/linux64/112.0.5615.49/chromedriver unexpectedly exited. Status code was: 127

Are you testing in docker (just want to make sure)?

@vmajor
Copy link

vmajor commented Apr 20, 2023

No, native in WSL2 in a conda venv. chrome fails with that error, firefox fails with "user profile not found" error even though it is meant to be headless and it runs fine with a gui when started manually. I will try to run Auto-GPT under windows next.

@universeroc
Copy link

Not works for me natively run on CentOS with conda env, how to fix it? help me thanks in advance:)

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.

8 participants