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 default GitHub assets #9573

Merged
merged 3 commits into from
Sep 25, 2022
Merged

Update default GitHub assets #9573

merged 3 commits into from
Sep 25, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Sep 25, 2022

Avoid intermittent CI fails.

Signed-off-by: Glenn Jocher [email protected]

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Streamlined asset naming and download process in Ultralytic's YOLOv5 GitHub repository.

πŸ“Š Key Changes

  • Simplified asset list generation with a list comprehension replacing a hard-coded list of model weights.
  • Removed the backup URL for downloading the assets, opting for a more straightforward download mechanism.

🎯 Purpose & Impact

  • The refactoring makes adding new model sizes or types more efficient πŸš€, improving code maintainability.
  • By removing the backup download URL, the download process is more straightforward, which may reduce complexity and potential errors 🧹.
  • Users will benefit from a cleaner and more reliable way to download necessary model files, improving user experience 🌐.

Signed-off-by: Glenn Jocher <[email protected]>
Signed-off-by: Glenn Jocher <[email protected]>
Signed-off-by: Glenn Jocher <[email protected]>
@glenn-jocher glenn-jocher merged commit 55fbac9 into master Sep 25, 2022
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch September 25, 2022 00:59
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