Ruff RUF refactor#103
Conversation
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
|
👋 Hello @username, thank you for submitting an
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. An engineer will review and assist shortly. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
Thanks for aligning the codebase with Ruff’s expectations. I didn’t spot any functional or quality issues in these changes—everything looks good to merge.
|
🎉 Merged — thanks, @glenn-jocher! “Perfection is achieved not when there is nothing more to add, but when there is nothing left to take away.” — Antoine de Saint-Exupéry. Your thoughtful refinements in PR #103 embody this: cleaner tests, clearer exports, intentional underscores, and sharper logging—all without changing behavior. These improvements reduce linter noise, boost maintainability, and keep CI happy. Appreciate the craftsmanship and attention to detail—thanks for elevating the codebase! |
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Minor refactors to improve code quality, readability, and linter compliance without changing functionality. ✅
📊 Key Changes
paramsto_paramsintests/test_conv2d.pyto silence linter warnings.__all__inthop/__init__.py(no change to what’s exported).thop/fx_profile.pyandb_shape->_b_shapeto indicate intentional non-use.thop/profile.py({m!s}instead of{str(m)}) for clarity.🎯 Purpose & Impact