Skip to content

Fix typo in Sony IMX500 doc#17871

Merged
glenn-jocher merged 2 commits intomainfrom
sony-doc-fix
Dec 1, 2024
Merged

Fix typo in Sony IMX500 doc#17871
glenn-jocher merged 2 commits intomainfrom
sony-doc-fix

Conversation

@lakshanthad
Copy link
Collaborator

@lakshanthad lakshanthad commented Nov 29, 2024

@glenn-jocher FYI

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Fixed a minor typo in the command-line example for Sony IMX500 integration documentation.

📊 Key Changes

  • Corrected a typo in the command for running YOLOv8 object detection. The incorrect dash () in --labels was replaced with the correct double-dash (--).

🎯 Purpose & Impact

  • 🛠 Improved Usability: Ensures users following the documentation can run the command without errors caused by the typo.
  • 📚 Enhanced Accuracy: Makes the integration documentation clearer and more reliable for implementation.

@UltralyticsAssistant UltralyticsAssistant added the documentation Improvements or additions to documentation label Nov 29, 2024
@UltralyticsAssistant
Copy link
Member

👋 Hello @lakshanthad, thank you for submitting an ultralytics/ultralytics 🚀 PR! 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/ultralytics 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! 🚀

Additional Notes:

If this PR addresses a bug, we kindly request a minimum reproducible example (MRE) if not already provided. This will help us verify and test the proposed changes. Please include code snippets, datasets, or other relevant details to ensure your fix can be thoroughly reviewed. 🐛🔍

Made with ❤️ by Ultralytics Actions

@lakshanthad lakshanthad added the TODO High priority items label Nov 29, 2024
@codecov
Copy link

codecov bot commented Nov 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.01%. Comparing base (6696e88) to head (c23c00b).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #17871   +/-   ##
=======================================
  Coverage   74.01%   74.01%           
=======================================
  Files         127      127           
  Lines       17098    17098           
=======================================
  Hits        12655    12655           
  Misses       4443     4443           
Flag Coverage Δ
Benchmarks 35.00% <ø> (ø)
GPU 39.13% <ø> (ø)
Tests 67.79% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glenn-jocher glenn-jocher merged commit 00fc95f into main Dec 1, 2024
@glenn-jocher glenn-jocher deleted the sony-doc-fix branch December 1, 2024 18:09
@glenn-jocher
Copy link
Member

@lakshanthad merged!

dariussingh pushed a commit to dariussingh/ultralytics that referenced this pull request Dec 11, 2024
@glenn-jocher glenn-jocher removed the TODO High priority items label Apr 27, 2025
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