Skip to content

Eliminate non-supported badges #25

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

Merged
merged 4 commits into from
Jan 7, 2025
Merged

Conversation

RizwanMunawar
Copy link
Member

@RizwanMunawar RizwanMunawar commented Jan 7, 2025

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

This PR updates several example notebooks by removing deprecated Kaggle and Gradient badges to streamline the interface while resetting all cell execution states for consistency.

📊 Key Changes

  • 🔗 Removed outdated "Run on Gradient" and "Open in Kaggle" badges from the headers of multiple notebooks.
  • 🔄 Reset all notebook cell execution counts to null for a fresh, unexecuted state.
  • 🧹 Improved overall notebook cleanliness by removing redundant blank lines.

🎯 Purpose & Impact

  • 🧼 Simplified Headers: Reduces clutter and ensures links only point to maintained resources like Colab.
  • 📋 Fresh Start for Usage: By resetting execution counts, the notebooks are more beginner-friendly and ready for users to execute step-by-step without existing outputs.
  • 🚀 Improved Accessibility: Focuses attention on Colab, Discord, and Forum links for better user guidance.

This update ensures a cleaner, more intuitive user experience in the notebooks. 😊

@UltralyticsAssistant UltralyticsAssistant added the documentation Improvements or additions to documentation label Jan 7, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @RizwanMunawar, thank you for submitting this ultralytics/notebooks 🚀 PR! We appreciate your contributions to improving the project. To ensure a smooth review process, please go through the checklist below and address any pending items:

  • Clearly Define the Purpose: Ensure your PR description explicitly outlines the purpose of the changes and links to any relevant issues if applicable.
  • Stay Up-to-Date: Confirm that your PR branch is synchronized with the main branch of the repository. If it's out of date, you can update it by clicking the 'Update branch' button or running git pull and git merge main locally.
  • Pass CI Checks: Verify that our Continuous Integration (CI) checks are passing. If any checks fail, please investigate and resolve the issues to ensure code quality.
  • Enhance Documentation: If your changes affect user workflows or features, update the relevant documentation.
  • Add or Update Tests: If relevant, incorporate test cases to cover your changes and confirm all tests pass successfully.
  • Sign the Contributor License Agreement (CLA): If this is your first contribution to Ultralytics, sign our Contributor License Agreement by commenting "I have read the CLA Document and I sign the CLA" in this pull request.
  • Keep It Focused: Ensure that changes are limited to the scope of this PR's purpose. Avoid introducing unrelated modifications to maintain concise and effective updates. Remember, “Simplicity is the ultimate sophistication” — Leonardo da Vinci.

📍 Next Steps

Your changes look promising! However, an Ultralytics engineer will review your contribution thoroughly and follow up soon. In the meantime, if you have any questions or need assistance, please leave a comment here. 🙌

For more details on contributing, check out the Contributing Guide.

Thank you for helping make Ultralytics better! 🌟

Copy link
Member

@ambitious-octopus ambitious-octopus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good!

@RizwanMunawar RizwanMunawar merged commit 6cb0592 into main Jan 7, 2025
2 checks passed
@RizwanMunawar RizwanMunawar deleted the remove-kaggle-paperspace branch January 7, 2025 16:43
@UltralyticsAssistant
Copy link
Member

🎉 Your PR has been merged! Huge thanks to @RizwanMunawar for leading this cleanup effort and to @ambitious-octopus for your valuable support. This impactful update streamlines our notebooks, making them more accessible and beginner-friendly. 🌟

As Marcus Aurelius wisely said, "What we do now echoes in eternity." Your contributions may seem small, but their ripple effects—simplifying learning, improving clarity, and enhancing user experiences—will undoubtedly inspire many. 🚀

Thank you for your attention to detail and commitment to progress. Together, we build tools that empower curiosity and discovery! 💡❤️

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants