Skip to content

Pin Docker-cpu FROM ubuntu:20.04#7677

Merged
glenn-jocher merged 1 commit intomasterfrom
docker-cpu
May 3, 2022
Merged

Pin Docker-cpu FROM ubuntu:20.04#7677
glenn-jocher merged 1 commit intomasterfrom
docker-cpu

Conversation

@glenn-jocher
Copy link
Copy Markdown
Member

@glenn-jocher glenn-jocher commented May 3, 2022

Pin FROM ubuntu:20.04

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Update and simplify the YOLOv5 CPU Dockerfile.

📊 Key Changes

  • 🚢 Switched the base Docker image from ubuntu:latest to ubuntu:20.04.
  • ✂️ Removed unnecessary steps for installing Python 3.9 since Ubuntu 20.04 comes with Python 3.8 by default.
  • 🔧 Changed python commands from python3.9 to the default python3.
  • 🧹 Cleaned up the installation process, removing more complex setup for a simpler one.

🎯 Purpose & Impact

  • 🔄 Ensures compatibility by using a more stable version of Ubuntu rather than the latest, which can vary over time.
  • ⚙️ Simplifies the Docker image by removing redundant installation steps for Python 3.9, as the version included with Ubuntu 20.04 is sufficient.
  • 👍 Improves the maintainability of the Dockerfile by using default python aliases, which reduces confusion and potential errors.
  • 💨 May result in faster build times and reduced complexity for users setting up the YOLOv5 environment on CPU-based systems.

@glenn-jocher glenn-jocher changed the title Update Docker-cpu Update Docker-cpu FROM ubuntu:20.04 May 3, 2022
@glenn-jocher glenn-jocher changed the title Update Docker-cpu FROM ubuntu:20.04 Pin Docker-cpu FROM ubuntu:20.04 May 3, 2022
@glenn-jocher glenn-jocher self-assigned this May 3, 2022
@glenn-jocher glenn-jocher merged commit e305aba into master May 3, 2022
@glenn-jocher glenn-jocher deleted the docker-cpu branch May 3, 2022 06:27
tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
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