Roboflow support

[SkalskiP]

Hi, @fcakyon 👋🏻!

Would you mind if I prepared a PR adding support for Roboflow datasets to your pip package?

[fcakyon]

Hello @SkalskiP it would be great! Feel free to open such PR 👍🏻

[SkalskiP]

Hi, @fcakyon 👋🏻! It took me a bit longer than expected, but here is the first PR that allows direct use of datasets from Roboflow Universe to train models. Let me know what you think.

#210

[fcakyon]

@SkalskiP, thanks for the PR! I have merged your PR to run roboflow cli tests, but there are several issues.

1. when the target dataset type is selected as yolov8, roboflow package requires ultralytics package to be installed in the env (which is not a yolov5-pip dependency): https://github.com/fcakyon/yolov5-pip/actions/runs/4022619564/jobs/6912550070
2. when the target dataset type is selected as yolov5, roboflow API returns a 404 error: https://github.com/fcakyon/yolov5-pip/actions/runs/4022749506/jobs/6912813939

Would you happen to have any advice to fix these issues?

PS: I am using this dataset in roboflow tests since its small: https://universe.roboflow.com/jacob-solawetz/aerial-maritime/dataset/10

[SkalskiP]

Hi, @fcakyon! I didn't see those issues yesterday. But of course, that needs to work flawlessly so you can count on my help. I'll start to work on a fix right away.

[SkalskiP]

Thank you very much for your help with CI. That is awesome that we can test automatically. 😎

[SkalskiP]

Hi, @fcakyon 👋🏻!

1. There is a minor bug in my PR that I'm going to fix. I always meant to download the dataset in yolov5 format for detection, not in yolov8.
2. It Looks like we have two bugs that you accidentally uncovered. I already created issues on our side:

• ModuleNotFoundError: No module named 'ultralytics' roboflow/roboflow-python#103
• HTTPError: HTTP Error 404: Not Found roboflow/roboflow-python#104

Interestingly that 404 error is thrown for https://universe.roboflow.com/jacob-solawetz/aerial-maritime/dataset/10 but not for other datasets I tested. That's why I propose using a different dataset in the CI/CD pipeline and let us debug that in parallel.

[SkalskiP]

Hi, @fcakyon 👋🏻!

I created the fix: #213. I tested all your CI/CD cases locally, and they all work. Your test suit fails on my PR now because I don't have access to your env.ROBOFLOW_TOKEN, and it looks like it's not being passed. :/ But it should work for you.

[fcakyon]

Ok lets merge and see how tests perform. Thanks for your efforts 👍

[SkalskiP]

Thanks for the opportunity and responsiveness.

[SkalskiP]

And yes, lets marge run CI. I'm happy to assist until it will work obviously :)

[fcakyon]

@SkalskiP it gives such error: https://github.com/fcakyon/yolov5-pip/actions/runs/4025808461/jobs/6919504263

[SkalskiP]

great! <3 Than I was thinking about adding deploy to roboflow

[fcakyon]

Nice! New version is live: https://github.com/fcakyon/yolov5-pip/releases/tag/7.0.8

[SkalskiP]

🚀 I'll share the news on LinkedIn tomorrow!

[SkalskiP]

@fcakyon btw, are you still in the US?

[fcakyon]

I am currently in Turkey.