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

Update val.py speed and study tasks #5608

Merged
merged 1 commit into from
Nov 10, 2021
Merged

Update val.py speed and study tasks #5608

merged 1 commit into from
Nov 10, 2021

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 10, 2021

Accepts all arguments now by default resolving #5600

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Refinement of validation benchmarks and study tasks in YOLOv5.

πŸ“Š Key Changes

  • Consolidated benchmarking and study tasks under a single conditional structure.
  • Enabled FP16 (opt.half = True) by default for the 'speed' task to enhance inference speed.
  • Set default confidence (opt.conf_thres) and intersection over union (opt.iou_thres) thresholds for the 'speed' task.
  • Streamlined the loops for processing multiple weights by directly iterating over them.
  • Simplified data output code to write results into files for the 'study' task.

🎯 Purpose & Impact

  • πŸš€ Simplifies the codebase and improves maintainability by reducing complexity in val.py.
  • ⏱ Enhances the performance (speed) of benchmarking tests by defaulting to FP16 precision, making benchmark results more reflective of real-world fast deployment scenarios.
  • βœ… Users can now more easily perform 'speed' and 'study' benchmarks with consistent and standardized settings.
  • πŸ“ˆ Improves the ease and consistency of capturing and reviewing results from model study benchmarks, aiding in research and development efforts.

Accepts all arguments now by default resolving #5600
@glenn-jocher glenn-jocher linked an issue Nov 10, 2021 that may be closed by this pull request
2 tasks
@glenn-jocher glenn-jocher self-assigned this Nov 10, 2021
@glenn-jocher glenn-jocher merged commit 30bc089 into master Nov 10, 2021
@glenn-jocher glenn-jocher deleted the update/val_tasks branch November 10, 2021 15:48
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
Accepts all arguments now by default resolving ultralytics#5600
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.

val.py --task speed parser parameter return nothing
1 participant