-
Notifications
You must be signed in to change notification settings - Fork 630
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
About Multi-Label NMS #175
Comments
@stigma0617 The previous implementation uses for loop in python to do multi-class NMS. In this PR, NMS for all classes is done together. If two bounding-boxes are with different classes, their overlap is set as 0. |
@tianzhi0549 Thanks for your reply :) |
@tianzhi0549 does multi-label NMS provide a mAP increase compared to single-label NMS, and if so how much? Thank you. |
Hey @tianzhi0549, I couldn't find the implementation of python Multi-Label NMS in your repo. Would you please where is your python Multi-Label NMS implementation? Thanks, |
@Auth0rM0rgan we've implemented this here: It lead to a mAP increase during inference on COCO. See ultralytics/yolov3#679 git clone https://github.com/ultralytics/yolov3
cd yolov3
python3 test.py --weights ultralytics68.pt --cfg yolov3-spp.cfg --img 608
|
@glenn-jocher thanks! Have you compare the performance of multi-label NMS with fast NMS, which introduced in yolact paper (I’m just curious)? thanks! |
@Auth0rM0rgan I've not used fast NMS before, but glancing at the arxiv summary of the yolact paper it seems that its a faster implementation with slightly lower mAP, though I don't know if it is primarily aimed at box nms or semantic segmentation nms, or both. In any case, they are not mutually exclusive. multi-class NMS merely replicates the same box with any class above threshold. |
Hi, it is awesome that the speed of FCOS is boosted by Multi-Label NMS.
I want to the details of Multi-Label NMS. Could you explain more details? or Is there any reference?
The text was updated successfully, but these errors were encountered: