Skip to content

Update Dockerfile #5343

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Gustav-Proxi
Copy link

This commit updates the Dockerfile to ensure compatibility with Detectron2 by upgrading the Python version from 3.6 to 3.8. The changes include: Base Image Change: Switched from nvidia/cuda:11.1.1-cudnn8-devel-ubuntu18.04 to nvidia/cuda:11.1.1-cudnn8-devel-ubuntu20.04 to provide a more recent environment with better package support. Python Installation: Removed the installation of Python 3.6 and replaced it with Python 3.8, ensuring that the python3 command points to the newly installed version.

Thanks for your contribution!

If you're sending a large PR (e.g., >100 lines),
please open an issue first about the feature / bug, and indicate how you want to contribute.

We do not always accept features.
See https://detectron2.readthedocs.io/notes/contributing.html#pull-requests about how we handle PRs.

Before submitting a PR, please run dev/linter.sh to lint the code.

This commit updates the Dockerfile to ensure compatibility with Detectron2 by upgrading the Python version from 3.6 to 3.8. The changes include:
Base Image Change: Switched from nvidia/cuda:11.1.1-cudnn8-devel-ubuntu18.04 to nvidia/cuda:11.1.1-cudnn8-devel-ubuntu20.04 to provide a more recent environment with better package support.
Python Installation: Removed the installation of Python 3.6 and replaced it with Python 3.8, ensuring that the python3 command points to the newly installed version.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 7, 2024
@Gustav-Proxi Gustav-Proxi marked this pull request as draft August 7, 2024 08:50
@Gustav-Proxi Gustav-Proxi marked this pull request as ready for review August 7, 2024 08:50
@Gustav-Proxi
Copy link
Author

hey requesting to change accept this change to python 3.8

@captain357
Copy link

Thank you so much. I was trying to solve the same issue, but it was a hassle to figure out the dependency versions. And glady thought of looking at PR.

Few minor changes I had to do for my setup was:
installing pip required distutils which was not found
RUN apt-get -y install python3-distutils

adding -f flag to ln since the python3 binary was already existing
ln -fsv /usr/bin/python3.8 /usr/bin/python3

Again, thank you for help :)

@Gustav-Proxi
Copy link
Author

glad to know it was of use to you, I'll include your recommendations and update the file so other are able to use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants