-
Notifications
You must be signed in to change notification settings - Fork 56
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
Does this repository compatible with yolov5 V3? #9
Comments
I just try, and it work. |
@zineos THank you for confirming... Just one thing is that, if I use custom model. the thing we should modified is only the number of class? Because my custom seems like getting a lot of bounding boxes when using with deepstream. |
Are you replacing yololayer.cu and hardswish.cu file in tensorrtx/yolov5 ? |
@zineos yes I have replace the hardswish.cu and yololayer.cu in tensorrtx/yolov5. I tested my model with the yolov5 detect.py seems like the model perform very well for example detecting person, it didn't show multiple box in same image. However, I got multiple boxes after converting to Deepstream app. |
If I use a custom model any parameter i should change? |
For now, I have no idea |
@zineos Thank you for help, I would like to find the problem and report here. |
@zineos The tensorrtx seems like remove the hardswish for V3, maybe this is the problem/ a commit in tensorrtx yolov5 hswish plugin replaced by x*hsigmoid 4980365 wang-xinyu [email protected] 2020/09/10 20:43 |
@luvwinnie Hi, Have you solved the issue about having a lot of bboxes. I am trying to make it work with yolov5 4.0 |
@KevinZhanggg This repository i think it need to use yolov5 version 2.0 or version 3.0/3.1 instead of version 4.0. Because it seems like the yolov5 changed some layers in the latest version. Everytime the yolov5 updated the cuda layer in this repository need to be updated. Btw, i haven't tested with version4.0 but use the version 3.0 instead. |
@luvwinnie yeah You are definitely right. It seems like some files have to be changed for yolov5 4.0. |
@luvwinnie Did you solve multiple bbox issue when you implement yolov5 3.0 ? I have the same error. |
@luvwinnie
Could you give me a hand? Thanks! |
@KevinZhanggg I'm not sure what was the problem however i retrained my model then the multiple boxes gone. of course you need to tune the NUM_CLASSES and also the class and nms confidence to make it work with your custom model. The most stable one is the model trained with evolved yaml file. @wxthss82 What do you mean "So I think if I use corrensponding v3 or v4 libmyplugins.so, we can use deepstream-app to get the same result, but failed." ? |
@luvwinnie I mean that if I use yolov5l v3 version of libmyplugins.so, I can use deepstream-app and get the correct result. But when I use yolov5l v4 version of libmyplugins.so, I get many boxes in deepstream-app. |
Hi, I have the same problem with yolov5-v.3, I run deepstream on 2 camera but I have multiple bounding boxes in the preview window. Can you tell me how to solve this issue please ? |
Seems like the yolov5 has released a V3 version, does this repository compatible with it?
The text was updated successfully, but these errors were encountered: