[jira] [Commented] (OPENNLP-494) Merging results from several name-finders

["Joern Kottmann (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OPENNLP-494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13246374#comment-13246374 ] 

Joern Kottmann commented on OPENNLP-494:
----------------------------------------

As said on the user list we should make a proxy TokenNameFinder which calls a set of name finder and outputs the merged names. Such a class will be useful for many users, because names usually must be merged to be used.
                
> Merging results from several name-finders
> -----------------------------------------
>
>                 Key: OPENNLP-494
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-494
>             Project: OpenNLP
>          Issue Type: New Feature
>          Components: Name Finder
>    Affects Versions: tools-1.5.3
>         Environment: Ubuntu oneiric x64 Java 7 update 3
>            Reporter: Jim Piliouras
>              Labels: patch
>             Fix For: tools-1.5.3
>
>         Attachments: TokenNameFinderEvaluator.java
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Made some small changes to the TokenNameFinderEvaluator class which hopefully allow merging of results from several name-finders. It just does that by calling the find method of all supplied name-finders. The only break is the fact that Java does not allow varargs anywhere but at the end of the argument list so i could not use "TokenNameFinder... namefinders" as the first parameter in the constructor - i had to pass an array instead. I think it's worth reversing the order of arguments  but that is a break too...

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira