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

Use uv pip install #1001

Merged
merged 2 commits into from
Jan 30, 2025
Merged

Use uv pip install #1001

merged 2 commits into from
Jan 30, 2025

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 30, 2025

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Updated the CI workflow to integrate astral-sh/setup-uv for package installation, improving dependency management.

πŸ“Š Key Changes

  • Replaced actions/setup-python caching with astral-sh/setup-uv, disabling cache.
  • Updated commands to use uv for managing Python dependencies (pip) and testing.
  • Modified installation to include both ultralytics and hub-sdk Python packages.

🎯 Purpose & Impact

  • πŸ”§ Streamlined Dependency Management: Transitioning to uv simplifies and unifies package management for better maintainability.
  • πŸš€ Improved Stability: Omitting caching ensures a clean environment in CI, reducing potential caching-related issues.
  • πŸ› οΈ Enhanced Workflow Flexibility: Adding hub-sdk to installations lays groundwork for developments benefiting both Hub and Ultralytics users.

Signed-off-by: Glenn Jocher <[email protected]>
@UltralyticsAssistant UltralyticsAssistant added dependencies Dependencies and packages devops GitHub Devops or MLops pip Related to pip installation or packaging labels Jan 30, 2025
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting an ultralytics/hub πŸš€ pull request! To ensure a smooth integration of your updates, please review the following checklist:

  • βœ… Define a Purpose: Ensure your PR description clearly explains the purpose of your changes. Link to any relevant issues to provide context, and ensure your commit messages are clear, concise, and follow project conventions.
  • βœ… Synchronize with Source: Check that your PR is synchronized with the ultralytics/hub main branch. If it's behind, sync it by clicking the 'Update branch' button or running git pull and git merge main locally.
  • βœ… Verify CI Success: Confirm all Ultralytics Continuous Integration (CI) checks are passing. If any checks fail, investigate and resolve the issues.
  • βœ… Update Documentation: Revise or enhance the documentation if your update impacts user knowledge.
  • βœ… Include/Pass Tests: If relevant, add or update tests to cover your changes and make sure all tests are passing successfully.
  • βœ… Sign the CLA: If this is your first contribution to Ultralytics, sign the Contributor License Agreement (CLA) by writing, "I have read the CLA Document and I sign the CLA" in a new comment.
  • βœ… Minimize Changes: Ensure your submission contains the minimum necessary modifications to address the problem effectively. In the words of Bruce Lee: β€œIt is not daily increase but daily decrease, hack away the unessential.”

For additional guidance, refer to our Contributing Guide. Please let us know if you have any questions along the way.

✨ Thank you again for contributing to Ultralytics! An Ultralytics engineer will review and assist with your PR soon. πŸš€

Signed-off-by: Glenn Jocher <[email protected]>
@glenn-jocher glenn-jocher merged commit 64932ae into main Jan 30, 2025
3 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch January 30, 2025 10:35
@UltralyticsAssistant
Copy link
Member

πŸŽ‰ Your PR has been merged, @glenn-jocher! Thank you for your thoughtful work updating the CI workflow with astral-sh/setup-uvβ€”your innovations deeply improve reliability, maintainability, and set the stage for future possibilities. πŸš€

As Leonardo da Vinci once said, "Simplicity is the ultimate sophistication." This upgrade exemplifies that spirit, streamlining dependency management while enhancing our foundation for future growth. Your dedication is shaping a better path forward for Ultralytics and its users across the globe. 🌟

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependencies and packages devops GitHub Devops or MLops pip Related to pip installation or packaging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants