Skip to content

Conversation

@RizwanMunawar
Copy link
Member

@RizwanMunawar RizwanMunawar commented Nov 5, 2025

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Adds a new KITTI training notebook, fixes IMX500 export naming to align with YOLO11, and refreshes the README notebooks table with new links and media. 🚀

📊 Key Changes

  • New notebook: Object Detection with KITTI using Ultralytics YOLO11, including training, prediction, and export examples.
  • README table updated:
    • Cleaned up formatting for improved readability.
    • Added entry for the KITTI notebook.
    • Construction-PPE notebook now includes a YouTube link and dataset link.
  • Consistency fix: Replaced incorrect IMX500 export model name yolov8n_imx_model/ with yolo11n_imx_model/ across multiple notebooks.
  • Notebooks data source updated (docs/notebooks-data.yml) to include:
    • Construction-PPE: YouTube + dataset links.
    • New KITTI entry with docs and dataset references.

🎯 Purpose & Impact

  • Improves user onboarding with a complete, ready-to-run KITTI tutorial for training YOLO11 on a well-known autonomous driving dataset. 🛣️
  • Ensures accurate export guidance for Sony IMX500 by aligning filenames with YOLO11, preventing confusion or export errors. ✅
  • Enhances discoverability and learning via enriched README table and added media links (YouTube, datasets). 🎥
  • Strengthens branding and documentation consistency across notebooks. 📚

RizwanMunawar and others added 3 commits November 5, 2025 12:08
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Added YouTube links and a new entry for KITTI detection.

Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
@RizwanMunawar RizwanMunawar self-assigned this Nov 5, 2025
@UltralyticsAssistant UltralyticsAssistant added detect Object Detection issues, PR's documentation Improvements or additions to documentation enhancement New feature or request labels Nov 5, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @RizwanMunawar, thank you for submitting a ultralytics/notebooks 🚀 PR! This is an automated review and an Ultralytics engineer will assist shortly. To ensure a seamless integration of your work, 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/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

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

📝 Quick review notes:

  • Consider adding YouTube/Blog links for the new KITTI notebook in docs/notebooks-data.yml when available.
  • Please double-check notebook text for minor inconsistencies (e.g., dataset link references and dataset split counts) to ensure everything aligns with the current KITTI docs and YAML.

Copy link
Member

@UltralyticsAssistant UltralyticsAssistant left a comment

Choose a reason for hiding this comment

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

🔍 PR Review

Made with ❤️ by Ultralytics Actions

Fix the Colab badge to open this KITTI notebook, correct the KITTI documentation link, and update the IMX500 export filename to the YOLO11 naming convention.

💬 Posted 3 inline comments

Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Copy link
Member

@UltralyticsAssistant UltralyticsAssistant left a comment

Choose a reason for hiding this comment

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

🔍 PR Review 2

Made with ❤️ by Ultralytics Actions

Everything runs as expected, but please update the stated KITTI validation-image count to 1,496 so the documentation matches the dataset that the notebook downloads and trains on.

💬 Posted 1 inline comment

Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Copy link
Member

@UltralyticsAssistant UltralyticsAssistant left a comment

Choose a reason for hiding this comment

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

🔍 PR Review 3

Made with ❤️ by Ultralytics Actions

Nice work shipping the KITTI walkthrough and surfacing it across README/docs. Please tidy the duplicated word in the new notebook intro and add a dataset link in docs/notebooks-data.yml so the autogenerated table shows a badge instead of a dash.

💬 Posted 2 inline comments

Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
RizwanMunawar and others added 2 commits November 5, 2025 13:00
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Copy link
Member

@UltralyticsAssistant UltralyticsAssistant left a comment

Choose a reason for hiding this comment

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

🔍 PR Review 4

Made with ❤️ by Ultralytics Actions

Replace the unsupported uv installation command with the standard pip install ultralytics so the new KITTI notebook runs cleanly across our hosted environments.

💬 Posted 1 inline comment

Copy link
Member

@lakshanthad lakshanthad left a comment

Choose a reason for hiding this comment

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

LGTM

@RizwanMunawar RizwanMunawar merged commit abd92b0 into main Nov 5, 2025
2 checks passed
@RizwanMunawar RizwanMunawar deleted the kitti-notebook branch November 5, 2025 16:12
@UltralyticsAssistant
Copy link
Member

🎉 Huge thanks for the merge on PR #78 in ultralytics/notebooks! @RizwanMunawar and @lakshanthad, your work shines.

“Alone we can do so little; together we can do so much.” — Helen Keller. Your collaboration delivered a polished KITTI training notebook for YOLO11, aligned IMX500 export naming to yolo11n_imx_model, and refreshed the README with clearer links and media—small details that create a big impact on onboarding and reliability.

This upgrade makes it easier for users to train, export, and learn—especially with the new KITTI tutorial and improved discoverability. For anyone getting started, see the Ultralytics KITTI dataset guide.

Appreciate the craftsmanship and consistency—onward and upward! 🚀

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

Labels

detect Object Detection issues, PR's documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants