Skip to content

Conversation

@Nick1200000
Copy link
Contributor

This PR introduces a Dockerfile that containerizes the application using the slim version of Python 3.10. It ensures a lightweight, clean, and production-ready environment for running the app.

📦 Key Features
Uses python:3.10-slim as the base image for minimal footprint.

Installs essential system packages (git, build-essential) for dependency builds.

Configures environment variables to prevent .pyc creation and enable unbuffered logging.

Installs Python dependencies from requirements.txt.

Copies application code into the container.

Creates necessary directories for logs and data cache.

Exposes port 7860 and runs the app using python webui/app.py.

🧪 Testing
✅ Verified the container builds successfully locally.

✅ Confirmed the application starts and runs as expected inside the container.

✅ Took screenshots to document successful execution.

Screenshot 2025-05-10 233445
Screenshot 2025-05-10 232744

@Nick1200000
Copy link
Contributor Author

Hi @maintainers, can you please check the PR and merge it if it matches your requirements. Please and Thanks

@ChenZiHong-Gavin ChenZiHong-Gavin self-assigned this May 12, 2025
@ChenZiHong-Gavin ChenZiHong-Gavin requested a review from Copilot May 12, 2025 05:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds a Dockerfile that containerizes the GraphGen application using python:3.10-slim. Key changes include:

  • Using a slim Python base image to minimize the footprint.
  • Installing essential system packages and Python dependencies.
  • Configuring environment variables, creating necessary directories, and exposing port 7860 to run the application.

Nick1200000 and others added 2 commits May 12, 2025 10:49
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Nick1200000
Copy link
Contributor Author

Hi @ChenZiHong-Gavin and @tpoisonooo , resolved the issue from the recommendation made by the Copilot, please merge it.

Removed the comment made by Github Copilot
@Nick1200000
Copy link
Contributor Author

Hi @ChenZiHong-Gavin @tpoisonooo @DancingJane @eveningdong , can anyone merge this one

@ChenZiHong-Gavin
Copy link
Collaborator

Hi @ChenZiHong-Gavin @tpoisonooo @DancingJane @eveningdong , can anyone merge this one

Hi, thanks for writing the Dockerfile. I'm testing it at the moment and will merge it as soon as the testing is done and everything is in order.

Resolve the issue of appuser
@Nick1200000
Copy link
Contributor Author

Nick1200000 commented May 12, 2025

image
Hi @ChenZiHong-Gavin , i had resolved the issue i was having a issue related addition of the appuser and now i had resolved the issue, from my side GraphGen is working fine

@Nick1200000
Copy link
Contributor Author

@ChenZiHong-Gavin , please check locally from the updated Dockerfile , please and Thank you

@ChenZiHong-Gavin ChenZiHong-Gavin merged commit 00a9d1f into open-sciencelab:main May 12, 2025
@ChenZiHong-Gavin
Copy link
Collaborator

merged, thank you for your contribution.

@ChenZiHong-Gavin ChenZiHong-Gavin mentioned this pull request May 12, 2025
@Nick1200000
Copy link
Contributor Author

@ChenZiHong-Gavin , can you close the issue for the Dockerfile

@Nick1200000
Copy link
Contributor Author

Subject: Contribution to GraphGen - Docker Integration and Request for Research Credit

Hi @ChenZiHong-Gavin , @tpoisonooo , @DancingJane , @eveningdong ,

I'm Nikhil, a DevSecOps engineer and a final-year student. I recently contributed to the GraphGen project by containerizing it with Docker (now merged into the main branch).

I came to know that there is ongoing research work or a publication being planned around GraphGen. Since my work enables containerized deployment for smoother experimentation and production use, I was wondering if it would be possible to include my name or contribution credit in any upcoming publication or related acknowledgments.

Please let me know if you'd like any further details about my contribution.

Best regards,
Nikhil Kumar

ChenZiHong-Gavin added a commit that referenced this pull request Jun 4, 2025
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.

2 participants