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

Update Dockerfile #5343

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

Conversation

Vaishakgkumar
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
@Vaishakgkumar Vaishakgkumar marked this pull request as draft August 7, 2024 08:50
@Vaishakgkumar Vaishakgkumar marked this pull request as ready for review August 7, 2024 08:50
@Vaishakgkumar
Copy link
Author

hey requesting to change accept this change to python 3.8

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.

2 participants