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

Simplify val.py benchmark mode with speed mode #9674

Merged
merged 1 commit into from
Oct 2, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Oct 2, 2022

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Optimization of validation task settings for benchmarking speed in YOLOv5.

πŸ“Š Key Changes

  • Renamed the task from benchmark to speed in benchmark.py for both segmentation and detection validation functions.
  • Simplified padding and rectangle inference conditions in segment/val.py and val.py by consolidating the operations into a single line.

🎯 Purpose & Impact

  • πŸ‘ The renaming of the task parameter clarifies the intent of the code, focusing specifically on speed benchmarking.
  • πŸ›  The code simplification helps to make the logic behind padding and shape inference easier to understand and maintain.
  • πŸš€ These changes may slightly improve the performance of the validation during speed benchmarks and provide more transparent metrics for users assessing model speed.

@glenn-jocher glenn-jocher self-assigned this Oct 2, 2022
@glenn-jocher glenn-jocher merged commit 1158a50 into master Oct 2, 2022
@glenn-jocher glenn-jocher deleted the update/speed branch October 2, 2022 11:37
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