mystic123 / tensorflow-yolo-v3

Implementation of YOLO v3 object detector in Tensorflow (TF-Slim)
https://medium.com/@pawekapica_31302/implementing-yolo-v3-in-tensorflow-tf-slim-c3c55ff59dbe
Apache License 2.0
890 stars 353 forks source link

NMS Error #84

Open cnmckee opened 5 years ago

cnmckee commented 5 years ago

You have an error in your non-max suppression implementation. You need to clamp your calculated dimensions of the bbox intersection at 0 (so they can't be negative) or else you can end up with a positive iou resulting from multiplying two negative dimensions. The positive iou may exceed the threshold even when the boxes don't intersect at all. This would happen if one bounding box fully sits both to the right and below another. I think this should fix the problems referenced in #40 and #78. Going to submit a PR