Fix possible image drawing issues

zhixuwei commented 4 days ago

I have read the CLA Document and I sign the CLA

🛠️ PR Summary

_{Made with ❤️ by Ultralytics Actions}

🌟 Summary

Improves image annotation logic in the plotting utility of YOLOv5.

📊 Key Changes

Adjusted the loop in the plot_images function to iterate consistently over the entire batch size (bs).

🎯 Purpose & Impact

Consistency in Annotations: Ensures that all images in the batch are consistently annotated, potentially increasing the reliability of visual output. 📷
Improved Visualization: Enhances the clarity and accuracy of annotations in image mosaics, which is beneficial for users evaluating model predictions. 📈

UltralyticsAssistant commented 4 days ago

👋 Hello @zhixuwei, thank you for submitting an ultralytics/yolov5 🚀 PR! This is an automated response, and an Ultralytics engineer will assist you soon. In the meantime, 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. Make sure your commit messages are clear and concise.
✅ Synchronize with Source: Verify your PR is synchronized with the ultralytics/yolov5 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: Ensure all Ultralytics Continuous Integration (CI) checks pass. If any checks fail, please address them.
✅ Update Documentation: Update the relevant documentation for any new features or modifications.
✅ Add Tests: If applicable, include or update tests to cover your changes, ensuring all tests pass successfully.
✅ Sign the CLA: Ensure you've signed our Contributor License Agreement by stating, "I have read the CLA Document and I sign the CLA" in the PR comments.
✅ Minimize Changes: Limit your changes to the minimum necessary. As Bruce Lee wisely said, "It is not daily increase but daily decrease, hack away the unessential."

For more guidance, please refer to our Contributing Guide. If your PR is related to a bug, please provide a minimum reproducible example (MRE) to aid the review process. If you have any questions, feel free to leave a comment. Thank you for contributing to Ultralytics! 🚀✨

zhixuwei commented 4 days ago

@glenn-jocher Simplify the loop conditions for this PR

glenn-jocher commented 2 days ago

@zhixuwei thank you, PR merged!

ultralytics / yolov5