Skip to content

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 9, 2022

@kalenmike @AyushExel adding ultralytics to Docker images and commented in requirements.txt

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Update to YOLOv5's Python package requirements and Docker build configurations.

πŸ“Š Key Changes

  • Added ipython, psutil, and thop to requirements, improving interactive sessions, resource monitoring, and performance computation.
  • Organized the requirements sections (base, logging, plotting, export, deploy, extras) for better readability.
  • Integrated ultralytics official package in Dockerfiles to facilitate better integration with Ultralytics' HUB services.
  • Ensured Pillow version compatibility by specifying it should be at least version 9.1.0.
  • Limited opencv-python version in Docker to prevent using versions beyond 4.5.4.68 due to potential compatibility issues.

🎯 Purpose & Impact

  • Enriches the development environment with tools for interactive testing (πŸ”) and system performance tracking (🏎️).
  • Categorization of the requirements.txt makes dependency management more intuitive and maintainable (πŸ› οΈ).
  • Dockerfile updates foster a more standardized development and deployment process by utilizing official Ultralytics packages (πŸ“¦).
  • Pillow and OpenCV version specifications safeguard against issues that may arise from future updates that are incompatible with YOLOv5 (πŸ›‘οΈ).

@glenn-jocher glenn-jocher self-assigned this Nov 9, 2022
@glenn-jocher glenn-jocher merged commit 86decb3 into master Nov 9, 2022
@glenn-jocher glenn-jocher deleted the ultralytics_pip_package branch November 9, 2022 14:05
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