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.

Sign up for GitHub

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

Jump to bottom

AutoAnchor and AutoBatch LOGGER #5635

Merged
merged 3 commits into from
Nov 13, 2021
Merged

AutoAnchor and AutoBatch LOGGER #5635

merged 3 commits into from
Nov 13, 2021

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 13, 2021 •
edited by UltralyticsAssistant
Loading

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Refactoring logs to improve readability and maintenance by using a unified logging system.

📊 Key Changes

  • Standardized logging: Replaced print statements with LOGGER.info or LOGGER.warning to use the centralized logging mechanism.
  • Added emoji support: Emojis can now be added to log messages to enhance user friendliness.
  • Enhanced feedback for anchor adjustments: More detailed info is provided when the anchor order is reversed or when new anchors are saved.
  • Updated error handling: Errors during anchor adjustments are now clearly logged.
  • Refactored computation of optimal batch size: Optimized batch size is now logged, and CUDA memory info is provided more succinctly.
  • Streamlined plots: Labels are plotted with an informant log message rather than a print statement.

🎯 Purpose & Impact

  • 🚀 Enhanced maintainability: Centralized logging simplifies modifications and maintenance of the logging functionality.
  • 📈 Improved diagnostics: A unified logging mechanism provides clearer and systematically structured logs.
  • 🛠 Easier debugging: Clear logged messages make it easier to identify issues with model performance or functionality.
  • 🎨 Better user experience: Use of emojis and structured messages makes reading logs more enjoyable and informative for users.

@glenn-jocher glenn-jocher self-assigned this Nov 13, 2021
@glenn-jocher glenn-jocher merged commit 80cfaf4 into master Nov 13, 2021
@glenn-jocher glenn-jocher deleted the update/logger branch November 13, 2021 14:40
@glenn-jocher glenn-jocher mentioned this pull request Feb 22, 2022
Merged
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
@glenn-jocher @pre-commit-ci
AutoAnchor and AutoBatch LOGGER (ultralytics#5635)
8a79868 
* AutoBatch, AutoAnchor `LOGGER`

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update autoanchor.py

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@glenn-jocher glenn-jocher

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@glenn-jocher