Skip to content
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

Update SAM2 notebook #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update SAM2 notebook #37

wants to merge 1 commit into from

Conversation

RizwanMunawar
Copy link
Member

@RizwanMunawar RizwanMunawar commented Mar 8, 2025

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Updated the Colab badge link in a notebook for a better user experience! πŸš€

πŸ“Š Key Changes

  • πŸ”— Replaced the Colab badge link to correctly reference the current notebook (inference-with-meta-sam-and-sam2-using-ultralytics-python-package.ipynb) instead of a different notebook.

🎯 Purpose & Impact

  • βœ”οΈ Improved Navigation: Ensures users are directed to the correct Colab notebook for inference scripts.
  • πŸ’‘ Ease of Use: Enhances user experience by making the notebook immediately accessible without confusion or broken links.
  • 🌍 Community Support: Keeps documentation up-to-date for seamless interaction and usage.

…age.ipynb

Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
@RizwanMunawar RizwanMunawar self-assigned this Mar 8, 2025
@UltralyticsAssistant UltralyticsAssistant added the documentation Improvements or additions to documentation label Mar 8, 2025
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @RizwanMunawar, thank you for submitting an ultralytics/notebooks πŸš€ PR! Your contribution is highly valuable, and we appreciate your effort in keeping the repository up-to-date. To ensure a smooth review process, 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. Clear commit messages adhering to conventions are essential for maintaining a clean history.
  • βœ… Synchronize with Source: Make sure your pull request is up-to-date with the latest main branch of the ultralytics/notebooks repository. If your branch is behind, please update it by clicking the 'Update branch' button or by running git pull and git merge main locally.
  • βœ… Ensure CI Checks Pass: Check that all Continuous Integration (CI) workflows, including tests, build, and linting, are passing. If any errors occur, please address them prior to final review.
  • βœ… Update Documentation: If relevant, update accompanying documentation to reflect your changes.
  • βœ… Add Tests: For modifications or new features, include or update tests to ensure functionality. Confirm that all tests are passing.
  • βœ… Sign the CLA: If this is your first contribution to Ultralytics, please sign our Contributor License Agreement (CLA). Leave a comment stating, "I have read the CLA Document and I sign the CLA" to confirm.
  • βœ… Minimize Changes: Ensure your changes are limited to what's necessary, avoiding unrelated edits. As Bruce Lee wisely said, "Hack away the unessential. The closer to the source, the less wastage there is." πŸ‘Š

For more detailed guidelines, refer to our Contributing Guide. Please let us know if you require further clarification or assistance.

🌟 Thank you for improving the user experience by updating the Colab badge link! Ensuring accurate and convenient navigation for our community is invaluable. An Ultralytics engineer will review this PR shortly and provide further feedback if needed. πŸš€

Keep up the amazing contributions! πŸ€—

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.

2 participants