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

IoU eps adjustment #10051

Merged
merged 1 commit into from
Nov 6, 2022
Merged

IoU eps adjustment #10051

merged 1 commit into from
Nov 6, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 6, 2022

Unify h1 and h2 with eps values

Signed-off-by: Glenn Jocher glenn.jocher@ultralytics.com

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhanced bbox_iou function for better intersection-over-union (IoU) calculations. πŸ“

πŸ“Š Key Changes

  • Adjusted width (w) and height (h) calculations by adding an epsilon (eps) to avoid division by zero.
  • Simplified the consistency term v in the Complete IoU (CIoU) calculation by removing unnecessary epsilons.

🎯 Purpose & Impact

  • πŸ› οΈ Prevention of potential division by zero errors when calculating bounding box dimensions, improving robustness.
  • πŸ“ˆ Increased numerical stability and precision of IoU-related metrics, which could lead to more reliable model evaluations and training.
  • 🧠 These updates help ensure the object detection is as accurate as possible, which is essential for applications relying on precise object localization.

Unify h1 and h2 with eps values

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher self-assigned this Nov 6, 2022
@glenn-jocher glenn-jocher changed the title IoU eps adjustment IoU eps adjustment Nov 6, 2022
@glenn-jocher glenn-jocher merged commit 78ed31c into master Nov 6, 2022
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-2 branch November 6, 2022 14:06
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.

None yet

1 participant