disable the assertion that requires density(y|x,z) <= density(y|x) wh…

…en using sgd since asynchrnous updates can legitimately break this assertion.
ldmt-muri · Apr 3, 2015 · b713d4a · b713d4a
1 parent e836ee6
commit b713d4a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/core/LatentCrfModel.cc b/core/LatentCrfModel.cc
@@ -1210,7 +1210,7 @@ bool LatentCrfModel::ComputeNllZGivenXAndLambdaGradientPerSentence(bool ignoreTh
 
   // update the log likelihood
   sentNll -= nLogZ;
-  if(nLogC < nLogZ) {
+  if(nLogC < nLogZ && learningInfo.optimizationMethod.subOptMethod->algorithm != SGD) {
     cerr << "this must be a bug. nLogC always be >= nLogZ. " << endl;
     cerr << "nLogC = " << nLogC << endl;
     cerr << "nLogZ = " << nLogZ << endl;