You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by jb...@apache.org on 2011/06/06 23:13:39 UTC

svn commit: r1132774 - /incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/perceptron/PerceptronModel.java

Author: jbaldrid
Date: Mon Jun  6 21:13:39 2011
New Revision: 1132774

URL: http://svn.apache.org/viewvc?rev=1132774&view=rev
Log:
Two loops for normalization now one. https://issues.apache.org/jira/browse/OPENNLP-154

Modified:
    incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/perceptron/PerceptronModel.java

Modified: incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/perceptron/PerceptronModel.java
URL: http://svn.apache.org/viewvc/incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/perceptron/PerceptronModel.java?rev=1132774&r1=1132773&r2=1132774&view=diff
==============================================================================
--- incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/perceptron/PerceptronModel.java (original)
+++ incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/perceptron/PerceptronModel.java Mon Jun  6 21:13:39 2011
@@ -106,12 +106,18 @@ public class PerceptronModel extends Abs
           maxPrior = Math.abs(prior[oid]);
       }
       
-      for (int oid = 0; oid < numOutcomes; oid++)
-        prior[oid] = Math.exp(prior[oid]/maxPrior);
-
       double normal = 0.0;
-      for (int oid = 0; oid < numOutcomes; oid++)
+      for (int oid = 0; oid < numOutcomes; oid++) {
+        prior[oid] = Math.exp(prior[oid]/maxPrior);
         normal += prior[oid];
+      }
+
+      //for (int oid = 0; oid < numOutcomes; oid++)
+      //  prior[oid] = Math.exp(prior[oid]/maxPrior);
+      //
+      //double normal = 0.0;
+      //for (int oid = 0; oid < numOutcomes; oid++)
+      //  normal += prior[oid];
 
       for (int oid = 0; oid < numOutcomes; oid++)
         prior[oid] /= normal;