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

Improve dev containers so autogpt can browse the web #1940

Closed

Conversation

bennyhobart
Copy link

Background

Currently dev containers do not contain everything necessary to run AutoGPT, and between different environments installing these dependencies can be quite a mess.

This diff aims to give the containers everything necessary to just follow the readme and get going with auto-gpt in dev containers

Changes

  • Update dockerfile to pull from know environment, the expectation is on apple silicon people will use rosetta to run this container, this change allows us to install chrome, sourcery and other standard things without much finangling
  • Install Chrome, necessary to browse
  • add options.add_argument("--no-sandbox") to Chrome startup so it can run in dev containers, this does have bad security implications https://chromium.googlesource.com/chromium/src/+/refs/heads/main/docs/design/sandbox.md, and we might be able to get around it in another way (changing user starting chrome from root -> userspace, moving from debian(?))

Documentation

All specific things added are self documented, I could add a comment above the chrome install line if we wanted to outline, but seems readable as is

Test Plan

Reload and rebuild dev container

Run autogpt with goals of [Navigate to https://en.wikipedia.org/wiki/John_Wick:_Chapter_4', 'summarize page', 'output to file'] and role of "an AI designed to summarize webpages"

Observe glory

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

Currently no tests exists for dev containers, I only learnt about these today when pulling down autogpt but might be wortwhile investing in

@bennyhobart
Copy link
Author

There's a few related issues for this
#1445
#1788
#1711

@bennyhobart bennyhobart deleted the devcontainer-fix2 branch April 16, 2023 16:46
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

1 participant