You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by jo...@apache.org on 2014/10/21 22:54:37 UTC
svn commit: r1633461 - in /opennlp/trunk/opennlp-tools/src:
main/java/opennlp/tools/ml/model/ComparableEvent.java
test/java/opennlp/tools/ml/perceptron/PerceptronPrepAttachTest.java
Author: joern
Date: Tue Oct 21 20:54:37 2014
New Revision: 1633461
URL: http://svn.apache.org/r1633461
Log:
OPENNLP-722 Ordering of features is removed from the Comparable Event. The feature odering should not depend on the ids assigned to them.
Modified:
opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/ml/model/ComparableEvent.java
opennlp/trunk/opennlp-tools/src/test/java/opennlp/tools/ml/perceptron/PerceptronPrepAttachTest.java
Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/ml/model/ComparableEvent.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/ml/model/ComparableEvent.java?rev=1633461&r1=1633460&r2=1633461&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/ml/model/ComparableEvent.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/ml/model/ComparableEvent.java Tue Oct 21 20:54:37 2014
@@ -35,11 +35,11 @@ public class ComparableEvent implements
public ComparableEvent(int oc, int[] pids, float[] values) {
outcome = oc;
- if (values == null) {
- Arrays.sort(pids);
- } else {
- sort(pids, values);
- }
+// if (values == null) {
+// Arrays.sort(pids);
+// } else {
+// sort(pids, values);
+// }
this.values = values; // needs to be sorted like pids
predIndexes = pids;
}
Modified: opennlp/trunk/opennlp-tools/src/test/java/opennlp/tools/ml/perceptron/PerceptronPrepAttachTest.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/test/java/opennlp/tools/ml/perceptron/PerceptronPrepAttachTest.java?rev=1633461&r1=1633460&r2=1633461&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/test/java/opennlp/tools/ml/perceptron/PerceptronPrepAttachTest.java (original)
+++ opennlp/trunk/opennlp-tools/src/test/java/opennlp/tools/ml/perceptron/PerceptronPrepAttachTest.java Tue Oct 21 20:54:37 2014
@@ -83,6 +83,6 @@ public class PerceptronPrepAttachTest {
MaxentModel model = TrainUtil.train(createTrainingStream(), trainParams, null);
- testModel(model, 0.7756870512503095);
+ testModel(model, 0.7791532557563754);
}
}