[Stefano Pellegrini] Fixed bug in BP[logdomain=1,inference=MAXPROD]
authorJoris Mooij <joris.mooij@tuebingen.mpg.de>
Mon, 22 Feb 2010 19:22:16 +0000 (20:22 +0100)
committerJoris Mooij <joris.mooij@tuebingen.mpg.de>
Mon, 22 Feb 2010 19:22:16 +0000 (20:22 +0100)
ChangeLog
src/bp.cpp

index 1c557e2..6a35501 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,7 @@
 git HEAD
 --------
 
+* [Stefano Pellegrini] Fixed bug in BP[logdomain=1,inference=MAXPROD]
 * Removed deprecated functionality
 * Updated ChangeLog for libDAI-0.2.4 (forgot a few entries)
 
index de26d1d..799043a 100644 (file)
@@ -512,7 +512,7 @@ std::vector<size_t> BP::findMaximum() const {
 
             // The allowed configuration is restrained according to the variables assigned so far:
             // pick the argmax amongst the allowed states
-            Real maxProb = numeric_limits<Real>::min();
+            Real maxProb = -numeric_limits<Real>::max();
             State maxState( factor(I).vars() );
             for( State s( factor(I).vars() ); s.valid(); ++s ){
                 // First, calculate whether this state is consistent with variables that