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

Standardize warnings with WARNING ⚠️ ... #9467

Merged
merged 3 commits into from
Sep 18, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

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

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Enhancements made to YOLOv5 codebase for improved stability and utility.

📊 Key Changes

  • Introduced emojis() function to standardize emoji use across different platforms, particularly for Windows.
  • Refined error messaging throughout the codebase to increase clarity and helpfulness.
  • Adjusted logging to be more informative and less repetitive, removing prefix "WARNING:" where redundant.
  • Incorporated TryExcept class to improve error handling in contexts and decorators.
  • Modified various functionalities for better compatibility and performance, such as dataset loading, automated batching, and image verification.

🎯 Purpose & Impact

  • 🔨 Purpose: These updates aim to create a more consistent user experience across different operating systems, reduce confusion with clearer error messages, and streamline the internal handling of exceptions and warnings.
  • ⚡️ Impact: Users can expect more descriptive and less cluttered logs, better system compatibility, and a smoother overall experience when using YOLOv5 functionalities. Developers benefit from increased code reliability and ease of debugging.

@glenn-jocher glenn-jocher self-assigned this Sep 18, 2022
@glenn-jocher glenn-jocher changed the title Standardize warnings Standardize warnings with WARNING ⚠️ ... Sep 18, 2022
@glenn-jocher glenn-jocher merged commit ca9c993 into master Sep 18, 2022
@glenn-jocher glenn-jocher deleted the update/warnings branch September 18, 2022 14:15
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