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

Add RKNN export format in notebooks #33

Merged
merged 15 commits into from
Jan 28, 2025
Merged

Conversation

RizwanMunawar
Copy link
Member

@RizwanMunawar RizwanMunawar commented Jan 28, 2025

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Updated titles and descriptions across multiple YOLO11 tutorial notebooks to better align them with their specific topics and added a new integration (RKNN) to the supported export formats.

πŸ“Š Key Changes

  • Updated notebook titles and introductions for better clarity and focus (e.g., "Object counting," "Heatmaps generation," "Brain-tumor detection," etc.).
  • Replaced vague generalizations like "Ultralytics YOLO11 notebook" with descriptions specific to the notebook's use case.
  • Adjusted terminology for functions (e.g., "track" instead of "count") where applicable.
  • Added RKNN integration as a supported export format in documentation tables.

🎯 Purpose & Impact

  • Purpose: To improve user experience by providing clear, use-case-specific notebook descriptions, making it easier for users to navigate and learn. Enhanced terminology (e.g., "track," "count") ensures technical precision. The RKNN addition broadens compatibility for model deployment.
  • Impact: Users can now better identify and utilize the most relevant notebooks. Clearer documentation reduces confusion and learning curves, making YOLO11 more accessible. RKNN support expands the toolbox for hardware-specific deployments. πŸš€

@UltralyticsAssistant UltralyticsAssistant added documentation Improvements or additions to documentation exports Model exports (ONNX, TensorRT, TFLite, etc.) labels Jan 28, 2025
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @RizwanMunawar, thank you for submitting a notebooks πŸš€ PR to Ultralytics! To ensure a smooth review process and successful integration of your updates, please review the following checklist:

  • βœ… Define a Purpose: Clearly explain the purpose of your update in your PR description, and link any relevant issues. Ensure your commit messages are clear, concise, and aligned with our conventions.
  • βœ… Synchronize with Source: Ensure your branch is up-to-date with the main branch of the ultralytics/notebooks repository. If needed, update it by clicking the 'Update branch' button or using git pull and git merge main locally.
  • βœ… Ensure CI Checks Pass: All Ultralytics Continuous Integration (CI) checks must pass. If any checks fail, kindly address the highlighted issues.
  • βœ… Update Documentation: Make sure corresponding documentation has been updated to reflect your changes, particularly if they include new features like RKNN format implementation.
  • βœ… Add Tests: If feasible, include or update relevant tests to validate the proposed changes, ensuring the updates work as expected.
  • βœ… Sign the CLA: If this is your first contribution to Ultralytics, sign the Contributor License Agreement (CLA) by commenting: I have read the CLA Document and I sign the CLA.
  • βœ… Minimize Changes: Focus your PR on the changes directly linked to your feature addition. For instance, in this case, ensure updates strictly pertain to adding RKNN export format and any associated clarifications/improvements in notebooks.

For further information, please consult our Contributing Guide to ensure alignment with our best practices.

πŸ’‘ This is an automated response to help streamline the PR review. An Ultralytics team member will personally review and assist with your PR shortly. If you have specific questions or require immediate support, please leave a comment below.

Thank you for your time and effort contributing to the enhancement of our notebooks! πŸŽ‰

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.

Looks good @RizwanMunawar !

@ambitious-octopus ambitious-octopus merged commit e433894 into main Jan 28, 2025
2 checks passed
@ambitious-octopus ambitious-octopus deleted the RizwanMunawar-patch-1 branch January 28, 2025 11:23
@UltralyticsAssistant
Copy link
Member

πŸŽ‰ Exciting news β€” this PR is officially merged! Huge thanks to @RizwanMunawar for leading the charge and @ambitious-octopus for the valuable contributions. Your updates bring clarity, precision, and broaden the impact of YOLO11 through the RKNN integrationβ€”an incredible step forward for the Ultralytics community! πŸš€

As Marcus Aurelius once said, β€œWhat we do now echoes in eternity.” Your thoughtful improvements will not only help users today but will resonate with countless others exploring the possibilities of YOLO11 for years to come. πŸ’‘βœ¨ Thank you for driving innovation and making a meaningful difference! πŸ™Œ

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 exports Model exports (ONNX, TensorRT, TFLite, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants