Skip to content

Conversation

otamajakusi
Copy link
Contributor

@otamajakusi otamajakusi commented Aug 30, 2022

About this PR

First of all thanks for providing the great repo!

This PR fixes the following issues:
#15
#7

original model is retrieved from
https://github.com/ultralytics/YOLOv5/releases/download/v6.1/YOLOv5n.pt
https://github.com/ultralytics/YOLOv5/releases/download/v6.1/YOLOv5s.pt
and converted on the original yolov5 repo https://github.com/ultralytics/yolov5 as follows:

python3 export.py --weights YOLOv5n.pt --include onnx
python3 export.py --weights YOLOv5s.pt --include onnx

@doleron
Copy link
Owner

doleron commented Sep 10, 2022

Hi @otamajakusi I'm busy as hell and cannot review your PR. I really apologize for it.

I'll check it as long I have enough time.

Thanks, Luiz.

@umloucobr
Copy link

Yes, I had this problem for 2 days and this solved it. When I ran the program it would throw a memory exception if the rows variable was higher than 19203. Thanks.

@DenisDoroshko
Copy link

thank you @otamajakusi this solved issue for me too (running on windows with opencv 4.6.0)

@doleron doleron merged commit a670deb into doleron:main Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants