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 time limit to 50 ms/img #7956

Merged
merged 1 commit into from
May 24, 2022
Merged

Increase NMS time limit to 50 ms/img #7956

merged 1 commit into from
May 24, 2022

Conversation

glenn-jocher
Copy link
Member

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

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved NMS time limit scaling with batch size.

πŸ“Š Key Changes

  • Adjusted the scaling factor for the NMS time limit from 0.03 to 0.05 per batch size (bs).

🎯 Purpose & Impact

  • Aimed at better performance scaling with larger batch sizes during Non-Maximum Suppression (NMS).
  • Potentially increases processing time allowance for NMS proportionally to batch size, leading to fewer early exits on larger batches.
  • Impact may be smoother performance for users with varying batch sizes, but could also result in slightly longer inference times on larger batches. πŸ•’πŸ’‘

@glenn-jocher glenn-jocher merged commit b5ebd47 into master May 24, 2022
@glenn-jocher glenn-jocher deleted the update/nms branch May 24, 2022 11:58
@glenn-jocher glenn-jocher self-assigned this May 24, 2022
tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
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