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() #11360

Merged
merged 1 commit into from
Apr 14, 2023
Merged

Update check_requirements() #11360

merged 1 commit into from
Apr 14, 2023

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 14, 2023

🤖 Generated by Copilot at d66d89d

Summary

✏️📝🐛

Fixed typos and inconsistencies in code and documentation. Updated check_requirements function in utils/general.py to use correct project name.

check_requirements
Fixing typos everywhere
Winter of cleanup

Walkthrough

  • Fix typos and inconsistencies in project name, log messages, and comments (link, F0

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Updating logging information for auto-installing YOLOv5 requirements.

📊 Key Changes

  • Corrected a log message that incorrectly referenced "YOLOv8" instead of "YOLOv5".

🎯 Purpose & Impact

  • 🛠️ Enhances clarity in log outputs, ensuring that users are correctly informed about actions relating to YOLOv5.
  • 💡 Prevents potential confusion about which version of the software the message pertains to, maintaining accuracy for users during auto-installation of requirements.
  • 👥 Affects users installing YOLOv5 with missing dependencies, as they'll now receive the correct software version name in notifications.

Signed-off-by: Glenn Jocher <[email protected]>
@glenn-jocher glenn-jocher merged commit 60e29e2 into master Apr 14, 2023
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch April 14, 2023 12:47
bandakopi pushed a commit to irajcode/yolov5 that referenced this pull request Jul 20, 2023
Update general.py

Signed-off-by: Glenn Jocher <[email protected]>
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