-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
CT Error #119
Comments
@glenn-jocher The issue with the model ID in the test data was causing the test cases to fail. I've updated the test data, so if you rerun the CT, it should work now. |
@yogendrasinghx yes, I see the latest scheduled CI from today is now passing. Nice work!!! |
@ |
@yogendrasinghx ah nevermind, I was looking at the HUB-SDK CI action, I thought you had placed the tests in there. Ok so what we want to do is take your new YAML that you created and place it in the HUB SDK CI, which is the reason this yaml exists in the first place. |
@yogendrasinghx the HUB SDK CI is the offical repo CI and follows exactly the template from Ultralytics, with all the correct run criteria like schedule, manual run capability etc, so the other yml that you have there is duplicating code and effort and adding confusion. |
@glenn-jocher Sure, I've added continuous testing to our existing CI pipeline and removed the ct.yml file. This will make things simpler. I've created a PR #120 for this; please check it out. |
@yogendrasinghx awesome, nice work on #120, closing this issue as completed :) |
Search before asking
HUB-SDK Component
Other
Bug
@yogendrasinghx 3 failing CI tests here introduced in #116
See https://github.com/ultralytics/hub-sdk/actions/runs/8650461977/job/23719232082
Environment
No response
Minimal Reproducible Example
No response
Additional
Also a few deprecation warnings here, but lower priority.
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: