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

Update Dockerfile #2663

Closed
wants to merge 1 commit into from
Closed

Conversation

diomabros
Copy link

As suggested in #2634 (comment)

Background

When i try to run Auto-GPT from docker (compose), it fail with the error in the link

Changes

As specified in the suggestion, adding this line "solve" the problem (I think that isn't optimal solution, but works)

Documentation

If there is a missing requirements, installing it "manually" solves the problem

Test Plan

I added this line to my cloned repo and it works

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

Copy link

@deb999983 deb999983 left a comment

Choose a reason for hiding this comment

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

This change shouldn't be done here, as there is already a requirements.txt file, and I can also see this dependency is also listed there, but it's below this line,
# Items below this point will not be included in the Docker Image,
and this command in Dockerfile,

RUN sed -i '/Items below this point will not be included in the Docker Image/,$d' requirements.txt && \
	pip install --no-cache-dir --user -r requirements.txt

removes it from the final image.

So a better fix would be to move the dependency, above that line, in the requirements.txt file, instead of tweaking the Dockerfile, something like

# OpenAI and Generic plugins import
openapi-python-client==0.13.4

# Items below this point will not be included in the Docker Image

@Pwuts
Copy link
Member

Pwuts commented Apr 20, 2023

Fixed in #2669

@Pwuts Pwuts closed this Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants