yolov5 icon indicating copy to clipboard operation
yolov5 copied to clipboard

translucent line for visibility

Open mucunwuxian opened this issue 1 year ago • 10 comments

Thank you 🙏 for your contribution to Ultralytics 🚀! Your effort in enhancing our repositories is greatly appreciated. To streamline the process and assist us in integrating your Pull Request (PR) effectively, please follow these steps:

  1. Check for Existing Contributions: Before submitting, kindly explore existing PRs to ensure your contribution is unique and complementary.

-> I checked that there are no similar PRs.



  1. Link Related Issues: If your PR addresses an open issue, please link it in your submission. This helps us better understand the context and impact of your contribution.

-> I created an issue. https://github.com/ultralytics/yolov5/issues/12612



  1. Elaborate Your Changes: Clearly articulate the purpose of your PR. Whether it's a bug fix or a new feature, a detailed description aids in a smoother integration process.

-> I added an argument to make the bounding box line translucent to improve visibility of results. How about this?



  1. Ultralytics Contributor License Agreement (CLA): To uphold the quality and integrity of our project, we require all contributors to sign the CLA. Please confirm your agreement by commenting below:

    I have read the CLA Document and I hereby sign the CLA

-> I have read the CLA Document and I hereby sign the CLA.



For more detailed guidance and best practices on contributing, refer to our ✅ Contributing Guide. Your adherence to these guidelines ensures a faster and more effective review process.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

📊 Key Changes

  • Imported the numpy library to manipulate image data.
  • Added a new parameter line_alpha for bounding box transparency in the detect.py script.

🎯 Purpose & Impact

The purpose of introducing line_alpha is to enhance the visibility of detection results by making the bounding boxes translucent. This change can help users discern background details that might be important for context without compromising the visibility of the bounding box itself. The impact is improved visual analysis for the user, especially in situations where seeing the background information is just as critical as seeing the object detections.

🌟 Summary

Introducing 🔎 translucent bounding boxes in Ultralytics' YOLOv5 for better visibility and context during object detection.

mucunwuxian avatar Jan 11 '24 00:01 mucunwuxian

@mucunwuxian hello! 👋

Thank you so much for your kind words and for your contribution to the YOLOv5 project! It's fantastic to see community members like yourself getting involved. 😊

Regarding your PR for adding a translucent bounding box line argument, that sounds like a potentially useful feature for improving result visualization. Please make sure to detail the changes in your PR description, including any new argument options and how they affect the output. This will help us understand and review your contribution more effectively.

Don't forget to link any related issues as mentioned, and if you haven't already, please sign the CLA to move forward with the contribution process.

Looking forward to reviewing your PR, and once again, thanks for supporting the YOLOv5 community! 🚀

glenn-jocher avatar Jan 11 '24 04:01 glenn-jocher

@glenn-jocher Hello!✋✨ Thank you for your reply!

I added comments. How abount this? Are there are any shortage?

mucunwuxian avatar Jan 11 '24 06:01 mucunwuxian

Hello @mucunwuxian! 👋

Great to hear that you've added comments to your PR. 👍

To ensure everything is in order, please double-check the following:

  • Your PR description should clearly explain the changes made and their impact on the codebase.
  • Make sure you've documented how to use the new translucent line feature, including any new argument syntax.
  • Ensure that your code changes are well-commented and follow the existing code style for consistency.
  • Verify that you've tested the new feature and it works as expected without introducing new bugs.

If all these points are covered and your PR is linked to the issue you created, there shouldn't be any shortage in your submission. Once everything looks good, we'll proceed with the review process.

Thank you for your diligence and for contributing to YOLOv5! 🌟

glenn-jocher avatar Jan 11 '24 12:01 glenn-jocher

@glenn-jocher Thank you for your reply!

There are only a few code changes. I think there is no problem.

It seems that the line break was added by the autocorrect function. I hope you can adjust the appearance in any way you like.

I think it will be more intuitive and user-friendly!

mucunwuxian avatar Jan 11 '24 13:01 mucunwuxian

Hello again @mucunwuxian! 😄

It's great to hear that the code changes are minimal and you've considered the user experience. If the autocorrect function has introduced unintended line breaks, it's a good idea to review and clean up the formatting before finalizing the PR. Clean and readable code is always appreciated!

Once you've made sure the appearance and functionality align with your intentions, we'll be happy to review it. Your efforts to make YOLOv5 more intuitive and user-friendly are very much appreciated!

Looking forward to your PR, and thank you for being proactive in enhancing the project! 🌟

glenn-jocher avatar Jan 11 '24 17:01 glenn-jocher

@glenn-jocher Thank you for your reply!

After arranging the appearance, I ran the automatic PR check again.👍 I hope you can check the contents.

mucunwuxian avatar Jan 12 '24 00:01 mucunwuxian

Hello @mucunwuxian,

Fantastic! Thank you for tidying up the appearance and rerunning the automatic checks. 👍

I'll take a look at the contents of your PR as soon as possible. Your proactive approach is greatly appreciated, and it's contributors like you who help make YOLOv5 better for everyone.

Thanks again for your contribution! 🌟

glenn-jocher avatar Jan 12 '24 02:01 glenn-jocher

Hi, @glenn-jocher Is there anything I can do for you?

mucunwuxian avatar Jan 29 '24 23:01 mucunwuxian

Hello @mucunwuxian,

Thank you for checking in! At this point, if you've ensured that your PR is complete with all necessary information and passes all checks, there's nothing more you need to do for now. I'll review your PR and get back to you with any feedback or if any further actions are required on your end.

Your patience and willingness to help are much appreciated! 😊 If you have any other questions or additional contributions, feel free to share them.

Have a great day!

glenn-jocher avatar Jan 30 '24 06:01 glenn-jocher


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I sign the CLA


2 out of 3 committers have signed the CLA.
:white_check_mark: (UltralyticsAssistant)[https://github.com/UltralyticsAssistant]
:white_check_mark: (glenn-jocher)[https://github.com/glenn-jocher]
:x: @mucunwuxian
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

github-actions[bot] avatar Apr 09 '24 12:04 github-actions[bot]