Skip to content

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 8, 2021

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Streamlining logging and warnings across YOLOv5 codebase.

πŸ“Š Key Changes

  • Removal of individual import logging in various files.
  • Consolidated logging setup by using a single LOGGER object from utils.general.
  • Changed all logging calls to LOGGER and updated warning levels when necessary.

🎯 Purpose & Impact

  • πŸš€ Enhances consistency: Standardized logging creates a more consistent and maintainable codebase.
  • πŸ› οΈ Eases maintenance: With a single source of logging configurations, modifying logging behavior becomes simpler.
  • 🐞 Improves debugging: A centralized logging approach helps in tracking issues more effectively.
  • πŸ”’ Reduces code clutter: Removing repetitive logging imports declutters the code.

πŸ‘©β€πŸ’»πŸ‘¨β€πŸ’» Users won't notice any direct functional changes, but developers will appreciate the cleaner and more maintainable logging system behind the scenes.

@glenn-jocher glenn-jocher merged commit 79bca2b into master Nov 8, 2021
@glenn-jocher glenn-jocher deleted the update/logger branch November 8, 2021 15:32
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* Logger consolidation

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

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant