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

detections and groundtruth are swapped in calculate_iou_partly() function #43

Open
GaneshAdam opened this issue Apr 19, 2023 · 1 comment

Comments

@GaneshAdam
Copy link

Hello Author,

Thanks for the repo!
I am wondering why detections and groundtruth are swapped. like while calling it is called as calculate_iou_partly(dt_annotations, gt_annotations, ..) and in function definiton it is used as calculate_iou_partly(gt_annotations, dt_annotations, ..).
The results are different if we keep same ordering.

@andraspalffy
Copy link
Member

Hello, thank you for your issue.

This is a known problem that we inherited from the kitti_eval repo:
https://github.com/traveller59/kitti-object-eval-python/blob/master/eval.py#L365

You can see that it was also swapped here. Since the output use of the variables is also swapped, and this function is about matching rectangles and calculating their overlaps, it should not matter.

@BalazsSzekeres can you have a look?

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

2 participants