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

LOGGER consolidation #5569

Merged
merged 2 commits into from
Nov 8, 2021
Merged

LOGGER consolidation #5569

merged 2 commits into from
Nov 8, 2021

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.

None yet

1 participant