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

Increase NMS fixed time limit 300ms + 30ms/img #7957

Merged
merged 2 commits into from
May 24, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 24, 2022

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Adjustment of the non-max suppression time limit in YOLOv5.

πŸ“Š Key Changes

  • Modified the time limit for non-max suppression from 0.1 + 0.05 * batch_size to 0.3 + 0.03 * batch_size.

🎯 Purpose & Impact

  • πŸ•’ Increases reliability: Adjusting the time limit allows more time for non-max suppression to operate, which could reduce the risk of prematurely terminating the process in scenarios with high numbers of detections.
  • πŸ“ˆ Improves performance for larger batches: The change could potentially improve detection quality for larger batch sizes without significantly increasing the time spent on each image.
  • πŸ” Reduces potential for missed detections: By allowing more time, particularly for larger images or batches, the update ensures that the algorithm has a sufficient window to identify and suppress overlapping bounding boxes, potentially increasing the accuracy of detections.

@glenn-jocher glenn-jocher changed the title Increase NMS fixed time limit to 200ms Increase NMS fixed time limit 2 May 24, 2022
@glenn-jocher glenn-jocher changed the title Increase NMS fixed time limit 2 Increase NMS fixed time limit 300ms + 30ms/img May 24, 2022
@glenn-jocher glenn-jocher merged commit 68ff6c9 into master May 24, 2022
@glenn-jocher glenn-jocher deleted the update/nms branch May 24, 2022 12:11
@glenn-jocher glenn-jocher self-assigned this May 24, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
* Increase NMS fixed time limit to 200ms

* Update general.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant