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

A few suggestions for your awesome project #8

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

vhvb1989
Copy link

@vhvb1989 vhvb1989 commented Jul 28, 2024

Watch your video and had a few ideas/suggestions for the repo.
It looks like some of them are also suggested in other PRs already (should've taken a look to existing PRs first :)

  • Adding devcontainer
  • Some updates to Dockerfile
  • Adding gunicorn as a production ready server instead of flask dev server

vhvb1989 and others added 6 commits July 27, 2024 20:16
- Use scissors icon instead of emoji in the title

- Changes to docker
  - Add .dockerignore to skip copping unwanted files to container
   - Use python:3.9-slim for a smaller container
   - Automatically download u2net.onx during container build
   - Use gunicorn to serve the app instead of the flask dev server

- Adding gunicorn to serve the flask app. This brings a production level server to host the app, instead of the flask dev server.

- Update Readme with notes about how to run with a container or locally

- Adding .devcontainer for Codespaces and/or locally working from WSL or a dev container.
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.

1 participant