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 check_requirements() single install #9353

Merged
merged 4 commits into from
Sep 10, 2022

Conversation

glenn-jocher
Copy link
Member

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

Faster install and better conflict resolution with single installation

Signed-off-by: Glenn Jocher glenn.jocher@ultralytics.com

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

This PR simplifies dependency checks and improves auto-update functionality in the Ultralytics YOLOv5 repository.

📊 Key Changes

  • Altered check_requirements function usage from requiring a tuple to a string for single package requirements.
  • Refined the way check_requirements handles package installation and error messaging.
  • Updated various files to use the new string input for single package checks.
  • Simplified the conditionals in check_requirements for handling different input types (path, list, string).

🎯 Purpose & Impact

  • Streamlined Code: Simplifies code by allowing single package names as strings rather than one-item tuples, making it clearer and cleaner.
  • Enhanced Auto-Update: Better handling of auto-updates for missing packages, providing a smoother user experience when dependencies are not met.
  • Clearer Messaging: Users receive more informative messages when packages are missing or automatically updated, reducing confusion and aiding in troubleshooting.

Faster install and better conflict resolution with single installation

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher self-assigned this Sep 9, 2022
@glenn-jocher glenn-jocher merged commit 24bf9cc into master Sep 10, 2022
@glenn-jocher glenn-jocher deleted the update/check_requirements branch September 10, 2022 08:11
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