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

ASF GitHub Bot commented on OPENNLP-1479:
-----------------------------------------

mawiesne commented on PR #559:
URL: https://github.com/apache/opennlp/pull/559#issuecomment-1846632031

   > Is there a spec for this behavior?
   
   That's an excellent question. Curiosity for the win! Maybe @jzonthemtn can 
comment on that. Both of you are English native speakers, so I think you can 
better judge whether to split apart those fragments or if they should be kept 
tied as a result of tokenization.
   
   For this finding, and after comments by Jeff, one should open a separate 
Jira, if this is a "topic" to work on.




> Write better tests for pattern verification (tokenizers)
> --------------------------------------------------------
>
>                 Key: OPENNLP-1479
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1479
>             Project: OpenNLP
>          Issue Type: Improvement
>          Components: Tokenizer
>    Affects Versions: 2.1.1
>            Reporter: Bruno P. Kinoshita
>            Assignee: Lara Marinov
>            Priority: Major
>             Fix For: 2.3.2
>
>
> From [https://github.com/apache/opennlp/pull/516#issuecomment-1455015772]
> At the moment our tests verify that the tokenizer objects are created 
> correctly (i.e. tests getters and setters, constructor, etc.), without 
> verifying the actual behavior when used in conjunction with other classes 
> (factory, tokenizer, trainers, etc).
> It would be best to test the patterns used in the factories for different 
> languages with some interesting sample data (maybe something from project 
> gutenberg, open source news sites, etc.).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to