Skip to content

Update coco_evaluation.py#1

Open
congyao wants to merge 1 commit intomasterfrom
congyao-patch-2
Open

Update coco_evaluation.py#1
congyao wants to merge 1 commit intomasterfrom
congyao-patch-2

Conversation

@congyao
Copy link
Owner

@congyao congyao commented Jun 7, 2016

In the original script, the matching procedure between ground truth rectangles and detection rectangles is problematic. To be exact, in line 53 to line 66, in each iteration the function seeks the best match for each ground truth rectangle and removes the best match from the list of detection rectangles, if any. However, the one actually removed is NOT the best match, but the last one in the list of detection rectangles, because of this line: eval_bboxes.remove(eval_box_id). Here, match instead of eval_box_id should be used, since eval_box_id becomes the last index after the loop.

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.

1 participant