Skip to content
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

ground truth value for iou #19

Open
JesseYang opened this issue May 5, 2017 · 5 comments
Open

ground truth value for iou #19

JesseYang opened this issue May 5, 2017 · 5 comments

Comments

@JesseYang
Copy link

In the code, the ground truth value for iou prediction is the iou between the anchor and the ground truth box (https://github.com/cory8249/yolo2-pytorch/blob/master/darknet.py#L111). But the paper says that " the objectness prediction still predicts the IOU of the ground truth and the proposed box ...". In my understanding the "proposed box" here is the predicted box, not the anchor box.

@longcw
Copy link
Owner

longcw commented May 5, 2017

Thank you for pointing out this.
I read the darknet code again and think that you are right. The darknet uses anchors to match gt_boxes and pred_boxes (https://github.com/pjreddie/darknet/blob/master/src/region_layer.c#L264) but uses ious between gt_boxes and pred_boxes as targets (https://github.com/pjreddie/darknet/blob/master/src/region_layer.c#L280).

I will fix this and try to train the network later.

@datlife
Copy link

datlife commented Jul 10, 2017

@longcw I am curious if the bug has been fixed. Thanks!

@acrosson
Copy link

acrosson commented Sep 5, 2017

Any updated on this?

@Haiyan-Chris-Wang
Copy link

@longcw is the bug fixed? And what mAP do you get now?

@YinlinHu
Copy link

YinlinHu commented Mar 5, 2018

The bug has been fixed very early at commit edda7d4

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

No branches or pull requests

6 participants