search

fangfangli / cleartk

Automatically exported from code.google.com/p/cleartk
0 stars 0 forks source link

OpenNLP parser wrapper adds nodes multiple times to CAS #243

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Leaf nodes are getting added twice to the CAS by the DefaultOutputTypesHelper, 
so that you'll find that the following fails (even when the first assert 
passes):

    Assert.assertEquals(3, npNode.getChildren().size());
    Assert.assertEquals(3, JCasUtil.selectCovered(this.jCas, TreebankNode.class, npNode).size());

Original issue reported on code.google.com by steven.b...@gmail.com on 14 Apr 2011 at 4:08

GoogleCodeExporter commented 9 years ago 
Fixed in r2851.

Original comment by steven.b...@gmail.com on 14 Apr 2011 at 4:13

GoogleCodeExporter commented 9 years ago 
good catch.  thanks for the fix.  i probably introduced this bug.  sorry!

Original comment by phi...@ogren.info on 15 Apr 2011 at 4:50