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

[FEATURE]: Docker containerization #1055

Open
zaidkhuroo opened this issue Jan 16, 2025 · 3 comments
Open

[FEATURE]: Docker containerization #1055

zaidkhuroo opened this issue Jan 16, 2025 · 3 comments
Labels
enhancement New feature or request

Comments

@zaidkhuroo
Copy link

Feature summary

Docker should've been used to containerize this project.

Feature description

I noticed that the current setup requires running pip install -r requirements.txt manually, and I'd like to suggest considering the use of Docker for containerizing the application. Here are some reasons:

1.Environment Consistency: This eliminates the "works on my machine" problem by encapsulating the application's dependencies and Python environment within a Docker container.

  1. Simplified Setup: With Docker, new contributors and users can start the application without worrying about setting up the environment manually.

I hope you find this suggestion helpful, I'd be happy to assist with the Dockerization process if needed.

Motivation

The need for Dockerization in this project arises from several critical aspects that can significantly enhance the development and deployment processes

Alternatives considered

No response

Additional context

No response

@zaidkhuroo zaidkhuroo added the enhancement New feature or request label Jan 16, 2025
@zaidkhuroo zaidkhuroo changed the title [FEATURE]: <Docker containerization> [FEATURE]: Docker containerization Jan 16, 2025
@rishsanyal
Copy link

Pushed a PR for this

@zaidkhuroo
Copy link
Author

as i installed all the packages through requirements.txt file, after running the main file i get the error message to install packages like selenium, openai... which are already in requirements. I've attached image with this comment.

Image

@LBC970
Copy link

LBC970 commented Jan 17, 2025

Isn't running a virtual environment inside a Docker container redundant?

For your previous post, I've run into issues where the Python interpreter calls Python 2 but the dependencies are installed for Python 3 (or vice versa) and I've had to specify which Python to run. To overcome this, I've also had some luck installing dependencies for a specific Python (3 vs 2).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants