Skip to content

Ultralytics Refactor https://ultralytics.com/actions#94

Merged
glenn-jocher merged 2 commits into
mainfrom
refactor-20250825024025
Aug 25, 2025
Merged

Ultralytics Refactor https://ultralytics.com/actions#94
glenn-jocher merged 2 commits into
mainfrom
refactor-20250825024025

Conversation

@pderrenger
Copy link
Copy Markdown
Member

@pderrenger pderrenger commented Aug 25, 2025

This Ultralytics PR refactors code to improve performance and readability. 🔄

Key changes include:

  • 🚀 Optimized various functions for faster execution.
  • 🧩 Simplified complex logic for better understanding and maintenance.
  • 🗑️ Removed redundant code to streamline operations.
  • 📚 Improved code structure and organization.

These changes aim to enhance the overall quality and efficiency of the code. 🌟

Learn more about Ultralytics:

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Documentation-only update that refreshes external links and download analytics targets for clearer, more reliable references. 🔗✨

📊 Key Changes

  • Updated download badge link from Pepy to a ClickHouse-based dashboard for more robust download analytics.
  • Switched PyTorch and TorchVision links to the official docs domain (docs.pytorch.org) for ResNet50, nn layers, and the Profiler.
  • Minor README cleanups to ensure consistent, up-to-date references.

🎯 Purpose & Impact

  • Improves link accuracy and reliability, reducing broken or outdated references. ✅
  • Provides better visibility into package downloads via a stable analytics page. 📈
  • Enhances developer experience with direct links to authoritative PyTorch documentation. 🧭
  • No API or behavior changes—safe to upgrade with zero impact on existing workflows. 🛡️

See the Ultralytics Docs for details: https://docs.ultralytics.com

@UltralyticsAssistant UltralyticsAssistant added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 25, 2025
@UltralyticsAssistant
Copy link
Copy Markdown
Member

👋 Hello @pderrenger, thank you for submitting an ultralytics/thop 🚀 PR! This is an automated response to help streamline the review process—an Ultralytics engineer will also be with you shortly for a deeper look. In the meantime, please review the following checklist:

  • Define a Purpose: Clearly explain the purpose of your fix or feature in your PR description, and link to any relevant issues. Ensure your commit messages are clear, concise, and adhere to the project's conventions.
  • Synchronize with Source: Confirm your PR is synchronized with the ultralytics/thop main branch. If it's behind, update it by clicking the 'Update branch' button or by running git pull and git merge main locally.
  • Ensure CI Checks Pass: Verify all Ultralytics Continuous Integration (CI) checks are passing. If any checks fail, please address the issues.
  • Update Documentation: Update the relevant documentation for any new or modified features.
  • Add Tests: If applicable, include or update tests to cover your changes, and confirm that all tests are passing.
  • Sign the CLA: Please ensure you have signed our Contributor License Agreement if this is your first Ultralytics PR by writing "I have read the CLA Document and I sign the CLA" in a new message.
  • Minimize Changes: Limit your changes to the minimum necessary 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

If this PR addresses a bug and an MRE (Minimum Reproducible Example) is not included, please add a short script or steps to reproduce the issue so we can validate quickly. 🧪

For more guidance, please refer to our Contributing Guide. Don’t hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀

PR Title: Ultralytics Refactor https://ultralytics.com/actions

PR Description:
This Ultralytics PR refactors code to improve performance and readability. 🔄

Key changes include:

  • 🚀 Optimized various functions for faster execution.
  • 🧩 Simplified complex logic for better understanding and maintenance.
  • 🗑️ Removed redundant code to streamline operations.
  • 📚 Improved code structure and organization.
    These changes aim to enhance the overall quality and efficiency of the code. 🌟

Learn more about Ultralytics:

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Docs and style refresh: updated badges/links to official PyTorch docs and analytics, plus minor import/order cleanups with no functional changes. ✨

📊 Key Changes

  • Updated README badges: downloads now link to a ClickHouse dashboard for analytics instead of Pepy. 📈
  • Switched several URLs to the official docs domains:
    • ResNet50: moved to docs.pytorch.org
    • PyTorch layers and profiler links: standardized to docs.pytorch.org
  • Minor code style improvements:
    • Reordered imports (isort-style) and removed extra blank lines in benchmarks/tests.

🎯 Purpose & Impact

  • More reliable, up-to-date documentation links for users. 🔗
  • Improved visibility into package download metrics via the new dashboard. 📊
  • Cleaner, more consistent codebase with linter-friendly imports. 🧹
  • No API or behavior changes—existing workflows are unaffected. ✅

See the Ultralytics Docs for details.

@glenn-jocher glenn-jocher merged commit dc74a2a into main Aug 25, 2025
5 checks passed
@glenn-jocher glenn-jocher deleted the refactor-20250825024025 branch August 25, 2025 00:51
@UltralyticsAssistant
Copy link
Copy Markdown
Member

🎉 Merged! Huge thanks to @pderrenger for the thoughtful docs refresh and to @glenn-jocher for the contributions. As Peter Drucker said, “What gets measured gets improved”—your shift to a robust ClickHouse-based download dashboard and updated links to official PyTorch docs make our references clearer, more reliable, and easier to maintain. No API changes, just a smoother developer experience and better visibility into adoption. Appreciate the polish—small improvements like this keep the ecosystem shining. See the Ultralytics Docs for details: https://docs.ultralytics.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants