Bugfix

[terezasoukupova]

Hello,
I think I have found a bug in your code. In pymot.py, line 238, where is the overlap computed? I think that there should be something like this before this line:
rect_groundtruth = Rect(groundtruths[gt_index])
rect_hypothesis = Rect(hypotheses[hypo_index])
overlap = rect_groundtruth.overlap(rect_hypothesis)

Am I right?

[DenisKuplyakov]

I also think it looks like a bug. But it's better to solve it this way, without intersecting rects another time:

self.total_overlap_ += 1.0 / munkres_matrix[gt_index][hypo_index]

Still surprised, that results I've get were looking adequate.

[herr-biber]

You are right. Thanks. Looks like a bug. I'd really appreciate a pull request and a small test example which reproduces the bug.

[terezasoukupova]

I am sorry, I am new to github and I do not know for what is exactly pull request useful. But another point, isn't it weird, that MOTA can be negative? E.g. when you have no misses, no mismatches and many false positives.

[herr-biber]

See https://help.github.com/articles/creating-a-pull-request/ on how to create a pull request. Basically you fork this repo. Commit changes to your fork and file the pull request to allow me to merge changes made in your fork to be merged into this master.

Negative MOTA is possible, yes.

[DenisKuplyakov]

@terezasoukupova has created pull request for this: #8