opennlp-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jim - FooBar();" <>
Subject Merging results from several name-finders...
Date Wed, 04 Apr 2012 14:10:56 GMT
On 04/04/12 14:57, Jörn Kottmann wrote:
> Please make a patch and attach it to some jira issue. You also need to 
> set the flag that you
> agree that the code will be licensed under AL 2.0. We don't accept 
> code contributions which
> are posted to the mailing list.
> More information on how to contribute can be found here:

Ok thanks i will...

> I had a quick look at your class. Wouldn't it be better write a 
> TokenNameFinder which
> acts like a proxy to a set of name finders and aggregates/aligns the 
> result of them?
> You can then pass an instance of this new name finder to the existing 
> evaluator. 

Yes, i can also do that but that sounds even more like a hack! At least 
by containing the issue inside TokenNameFinderEvaluator we're saying:

*"Look this is an evaluation issue and has little to do with the 

otherwise we're saying

*"Hmmm, there is a different, better name-finder and all it does that 
calls the regular ones!"*

That is slightly misleading isn't it?
The way i set it up we can easily write some  code inside the existing 
TokenNameFinder class adding some "if"s to handle the case where the 
user passes in some more name-finders. IT doesn't even have to break 
backwards compatibility...Unless you insist i follow your suggestion, I 
will follow my rationale but improve on the code so it properly handles 
all cases inside the existing TokenNameFinder class and open a jira to 
submit it...


  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message