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

Download models #50

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Conversation

john-rocky
Copy link
Contributor

@john-rocky john-rocky commented Jul 31, 2024

CoreML models are automatically downloaded in the background.
Tapping on a model will prioritize its download.
Downloaded models are automatically compiled and made available to users.
The URL is temporary.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

WARNING ⚠️ this PR is very large, summary may not cover all changes.

🌟 Summary

This PR updates the Ultralytics YOLO iOS app project files to improve model management and build processes.

📊 Key Changes

  • Added additional model files and updated CoreML model packaging.
  • Integrated ZIPFoundation dependency for handling model file zipping/unzipping.
  • Improved project settings and configuration in Xcode.
  • Enhanced the README instructions for cloning the repository, opening the project, and adding models.

🎯 Purpose & Impact

  • Enhanced Model Management: The added model files and updated packaging streamline the integration and usage of YOLOv8 models within the iOS app, improving performance and ease of use.
  • Simplified Setup: Updated README and project settings simplify the developer setup process, making it easier to get started with the YOLO iOS app.
  • Efficiency: Integration of ZIPFoundation allows efficient model file management, reducing file size and potentially improving app performance.

These changes make the development experience smoother and improve the app's readiness and reliability when adding new models or preparing for deployment. 🚀🔧

@glenn-jocher
Copy link
Member

@john-rocky this is good!

We want to keep models out of the codebase though (as in main branch), since otherwise subsequent changes will make the git chain very long and heavy. Can you remove the YOLOv8m CoreML model from this PR?

@john-rocky
Copy link
Contributor Author

I removed Yolov8m coreml model from code.
The app download it.

@glenn-jocher
Copy link
Member

@john-rocky awesome!

@glenn-jocher glenn-jocher added the TODO High priority items label Aug 11, 2024
@glenn-jocher glenn-jocher changed the title Downlod models Download models Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TODO High priority items
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants