fix: prevent logging config clobbering

[rkechols]

Fix for issue: #10132

• Previous behavior: loading this repository with torch.hub.load clobbers the existing logging configuration by modifying the root logger's configuration.
• New behavior: loading this repository with torch.hub.load only clobbers the logging configuration for logger yolov5 and its descendants.

🛠️ PR Summary

_{Made with ❤️ by Ultralytics Actions}

🌟 Summary

Enhanced logging initiation process with dynamic logger name setup in the YOLOv5 code.

📊 Key Changes

• Moved the "yolov5" logger name into a variable before initializing the logging.
• Called set_logging() with the new logger_name variable.

🎯 Purpose & Impact

• 🛠️ Maintainability: Defining the logger name as a variable makes the code more maintainable and clear.
• 🔍 Clarity: It's now easier to change the logger name in one place if needed in the future.
• 👥 User Experience: No direct impact on end users, but developers will appreciate the cleaner setup for logging.

[github-actions]

👋 Hello @rkechols, thank you for submitting a YOLOv5 🚀 PR! To allow your work to be integrated as seamlessly as possible, we advise you to:

• ✅ Verify your PR is up-to-date with ultralytics/yolov5 master branch. If your PR is behind you can update your code by clicking the 'Update branch' button or by running git pull and git merge master locally.
• ✅ Verify all YOLOv5 Continuous Integration (CI) checks are passing.
• ✅ Reduce changes to the absolute minimum required for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." — Bruce Lee

[ghost]

Please what problem this line 129 and like 130 had??

[ghost]

Oooh okay I get it now. You run before defining. Thank you. I was confused at first. 😂😂😂

[rkechols]

@Christiansarpong , the linked issue, #10132, has more details if needed.

[glenn-jocher]

@rkechols thanks for the PR! Can you please update your branch to master and then I'll merge?

[rkechols]

@glenn-jocher it should be good now!
Thanks!

[glenn-jocher]

@rkechols thanks, but for some reason GitHub is showing me an out of date message here. It's strange because usually there's a button to update. I think there may be something wrong with the PR. Can you please close this one and open a new one with your updates? I'll prioritize that and should be able to merge immediately. Thanks!

EDIT: maybe it's because I just merged another PR. Can you merge again? Apologies, just want to be on the safe side.

[rkechols]

Yes, there was another button to update from main. It's running now.

[rkechols]

@glenn-jocher , ready for you

[glenn-jocher]

@rkechols PR is merged. Thank you for your contributions to YOLOv5 🚀 and Vision AI ⭐