Skip to content

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

Merged
glenn-jocher merged 1 commit intomainfrom
refactor-20250825024025
Aug 25, 2025
Merged

Ultralytics Refactor https://ultralytics.com/actions#74
glenn-jocher merged 1 commit intomainfrom
refactor-20250825024025

Conversation

@pderrenger
Copy link
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

Polishes notebooks and project docs, standardizes a GitHub Action, and updates the downloads badge to a new analytics dashboard — bringing cleaner links and more reliable stats 📈✨

📊 Key Changes

  • GitHub Actions: Minor formatting and quoting cleanup in .github/workflows/table.yml for consistency.
  • README: Standardized language selector URLs by adding trailing slashes for most locales.
  • Notebooks: Updated the “Downloads” badge links across multiple notebooks to point to a new ClickHouse-based dashboard instead of pepy.tech.

🎯 Purpose & Impact

  • Improved consistency and readability in CI configuration, reducing formatting noise for future edits.
  • Cleaner, more reliable international doc links for users navigating translations.
  • More robust and likely faster download metrics via the new analytics dashboard, helping users and maintainers track adoption more accurately.

See the Ultralytics Docs for details.

@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops documentation Improvements or additions to documentation labels Aug 25, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @pderrenger, thanks for opening an ultralytics/notebooks 🚀 PR! This is an automated response to help move things along quickly—an Ultralytics engineer will review and assist soon. In the meantime, please review the checklist below to ensure a smooth merge:

  • ✅ 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/notebooks 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

Additional notes tailored to your PR:

  • Thanks for the workflow YAML cleanup and consistency updates—nice touch on indentation and quoting 🎯
  • The README locale link normalization looks good; please double-check any locales without trailing slashes for consistency
  • The notebooks’ “Downloads” badge target switched to the new analytics dashboard—please confirm the URL appears correctly in all updated notebooks and that no execution outputs or metadata were unintentionally changed
  • If any notebooks were re-saved, ensure outputs are cleared and metadata noise is minimized to keep diffs clean ✨
  • If this refactor includes changes that could affect end users, consider a brief note in docs or notebook headers to clarify the update

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! 🚀

@glenn-jocher glenn-jocher merged commit 7edd1c9 into main Aug 25, 2025
1 check passed
@glenn-jocher glenn-jocher deleted the refactor-20250825024025 branch August 25, 2025 00:45
@UltralyticsAssistant
Copy link
Member

Merged with gratitude! Thank you, @pderrenger, for the thoughtful polish across notebooks and docs, and kudos to @glenn-jocher for the contributions. As Peter Drucker said, “What gets measured gets managed.” Your update to the ClickHouse-powered downloads dashboard, cleaner international links, and standardized workflow make our ecosystem more reliable, measurable, and welcoming for the community.

Appreciate the attention to detail—this raises the bar for consistency and clarity across Ultralytics. See the Ultralytics Docs for details: https://docs.ultralytics.com

Onward and upward! 🚀

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

Labels

devops GitHub Devops or MLops documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants