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 ci.yaml for Classify-first benchmarking #9949

Merged
merged 3 commits into from Apr 10, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 10, 2024

More likely to succeed with less exotic layers.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Introduction of an improved benchmarking sequence in CI workflow πŸš€

πŸ“Š Key Changes

  • Reordered benchmarking steps for better workflow efficiency.
  • Specifically, the benchmarking for ClassificationModel has been moved up for earlier execution.

🎯 Purpose & Impact

  • Efficiency Boost: This reordering aims to optimize the Continuous Integration (CI) process, potentially leading to quicker feedback during development phases. πŸ•’
  • Enhanced Coverage: By adjusting the order in which models are benchmarked, the change might also help in identifying model-specific issues more rapidly, contributing to an overall improvement in code quality and application performance. πŸ”
  • Potential User Impact: For both developers and users of Ultralytics software, this could mean faster release cycles, less waiting time for new features or bug fixes, and a more stable product experience. ✨

πŸ“ˆ Every little improvement in the development process can lead to significant benefits in software stability and performance, making this change, although potentially small in appearance, quite impactful in the grand scheme of things!

More likely to succeed with less exotic layers.
Copy link

codecov bot commented Apr 10, 2024

Codecov Report

All modified and coverable lines are covered by tests βœ…

Project coverage is 76.12%. Comparing base (ea03db9) to head (4e37bd4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9949   +/-   ##
=======================================
  Coverage   76.12%   76.12%           
=======================================
  Files         121      121           
  Lines       15294    15294           
=======================================
  Hits        11643    11643           
  Misses       3651     3651           
Flag Coverage Ξ”
Benchmarks 36.05% <ΓΈ> (ΓΈ)
GPU 38.03% <ΓΈ> (+0.01%) ⬆️
Tests 71.47% <ΓΈ> (ΓΈ)

Flags with carried forward coverage won't be shown. Click here to find out more.

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@glenn-jocher glenn-jocher merged commit ebc9df5 into main Apr 10, 2024
5 of 6 checks passed
@glenn-jocher glenn-jocher deleted the benchmark-classify-first branch April 10, 2024 16:19
hmurari pushed a commit to hmurari/ultralytics that referenced this pull request Apr 17, 2024
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