search

saebyn / munkres-cpp

Kuhn-Munkres (Hungarian) Algorithm in C++
http://saebyn.info/2007/05/22/munkres-code-v2/
GNU General Public License v2.0
204 stars 79 forks source link

Wrong result of function replace_infinites #12

Closed Gluttton closed 10 years ago

Gluttton commented 10 years ago

Based on tests: MunkresTest.replace_infinites_4x4Case001_Success, MunkresTest.replace_infinites_4x4Case002_Success, MunkresTest.replace_infinites_4x4Case003_Success and MunkresTest.replace_infinites_4x4Case004_Success result of function replace_infinites is wrong.

It is looks like real bug (when determined min it not checked on equality to infinity, so max became infinity).

saebyn commented 10 years ago

Yep, I've already got this fixed. Thank you for letting me know :)

Gluttton commented 10 years ago

Thank you for letting me know :)

No problem, it's pleasure for me :) !

P.S. I think when you push fixes you will close this issue.