-
-
Notifications
You must be signed in to change notification settings - Fork 8k
[EXP] Adds uv run pytest
for tests
#17663
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
Conversation
👋 Hello @Burhan-Q, thank you for submitting your
For more detailed guidelines, check out our Contributing Guide. Feel free to leave a comment if you need further assistance. Thank you once again for contributing to Ultralytics! 🚀 |
Interesting failure with the key take away: Specifically:
Full error
|
Even deeper error than previously. This one was a lot more to parse. The conflict is between FYI @glenn-jocher @Laughing-q @ambitious-octopus Full error
|
@Burhan-Q yes, also got another error on |
Yeah @glenn-jocher the issue here was also with Numpy conflicts, but as a transient requirements for other packages. At first it was due to I just wanted to point this out since I previously had mentioned sunsetting 3.8 support previously. Not saying this should instigate an immedate change, but instead maybe a chance to start planning and give users sufficent notice of the change. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17663 +/- ##
===========================================
- Coverage 78.40% 34.98% -43.42%
===========================================
Files 129 129
Lines 17227 17227
===========================================
- Hits 13506 6027 -7479
- Misses 3721 11200 +7479
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Thank you for highlighting this. The conflicts with |
Thank you for sharing this. For assistance with YOLO implementation or troubleshooting, we recommend reviewing our YOLO Common Issues guide which covers many common scenarios. If you're experiencing a specific error or need implementation guidance, please share details about your use case and we'll help point you to the relevant documentation resources. |
Trial to see if using
uv
to invokepytests
would be beneficial for running CI tests.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Simplified command execution in CI by using
uv run
.📊 Key Changes
pytest
command withuv run pytest
in the GitHub Actions CI workflow.🎯 Purpose & Impact
uv run
for command running, which could unify or simplify environment management.