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

feat: Update Dockerfile of Thumbor to install opencv #2229

Closed
wants to merge 1 commit into from

Conversation

rrchai
Copy link
Contributor

@rrchai rrchai commented Oct 13, 2023

Note
"The default beeyev/thumbor-s3:7.4-alpine image does not include the opencv package, which is required for smart detection. This PR is being created to document the work to install opencv in the image, and it will be closed soon."

Below config is also added to .env just in case, but I think it's default.

DETECTORS=['thumbor.detectors.feature_detector','thumbor.detectors.face_detector']

@rrchai rrchai self-assigned this Oct 13, 2023
@rrchai rrchai changed the title feat: Update Dockerfile of Thumbor to install opencv feat: Update Dockerfile of Thumbor to install opencv lib Oct 13, 2023
@rrchai rrchai changed the title feat: Update Dockerfile of Thumbor to install opencv lib feat: Update Dockerfile of Thumbor to install opencv Oct 13, 2023
@rrchai rrchai closed this Oct 13, 2023
@rrchai rrchai deleted the fit-org-avatar branch October 13, 2023 13:31
@tschaffter
Copy link
Member

@rrchai This post-task documentation work could have been done directly in the ticket instead of opening and closing a PR, which triggers CI workflows.

RUN apk update \
    && apk --no-cache --update add build-base 
RUN pip install --upgrade pip \
    && pip install scikit-build cmake \
    && pip install --no-use-pep517 opencv-python --no-cache-dir

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