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 2012/03/15 15:55:22 UTC

svn commit: r1301021 - /opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/NameSampleDataStreamFactory.java

Author: joern
Date: Thu Mar 15 14:55:21 2012
New Revision: 1301021

URL: http://svn.apache.org/viewvc?rev=1301021&view=rev
Log:
OPENNLP-474 Fixed broken cross validation.

Modified:
    opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/NameSampleDataStreamFactory.java

Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/NameSampleDataStreamFactory.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/NameSampleDataStreamFactory.java?rev=1301021&r1=1301020&r2=1301021&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/NameSampleDataStreamFactory.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/NameSampleDataStreamFactory.java Thu Mar 15 14:55:21 2012
@@ -17,7 +17,7 @@
 
 package opennlp.tools.formats;
 
-import java.io.InputStreamReader;
+import java.io.FileInputStream;
 
 import opennlp.tools.cmdline.ArgumentParser;
 import opennlp.tools.cmdline.CmdLineUtil;
@@ -50,10 +50,11 @@ public class NameSampleDataStreamFactory
     language = params.getLang();
 
     CmdLineUtil.checkInputFile("Data", params.getData());
+    
+    FileInputStream sampleDataIn = CmdLineUtil.openInFile(params.getData());
 
-    ObjectStream<String> lineStream;
-    lineStream = new PlainTextByLineStream(new InputStreamReader(
-        CmdLineUtil.openInFile(params.getData()), params.getEncoding()));
+    ObjectStream<String> lineStream = new PlainTextByLineStream(sampleDataIn.getChannel(),
+        params.getEncoding());
 
     return new NameSampleDataStream(lineStream);
   }