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

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

rzo1 commented on PR #154:
URL: https://github.com/apache/opennlp-sandbox/pull/154#issuecomment-2219615339

   Wondering if checkstyle is active for sandbox? Might be good to have it for 
future PRs.




> Provide tests for MetaSummarizer
> --------------------------------
>
>                 Key: OPENNLP-1592
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1592
>             Project: OpenNLP
>          Issue Type: Test
>          Components: Summarizer
>    Affects Versions: 2.3.3
>            Reporter: Martin Wiesner
>            Assignee: Martin Wiesner
>            Priority: Major
>             Fix For: 2.4.0
>
>
> The summarizer (sandbox) component lacks Tests for MetaSummarizer.java.
> Currently, only an orphaned main method exists in the corresponding class. 
> Therefore, it is much desirable to have a fix set of tests that demonstrate 
> this class actually works (Note: it doesn't as is).
> Aims:
>  * Provide a minimal set of tests for MetaSummarizer
>  * Improve the existing code so that summarization works (again)
>  * Improve the code quality
>  * Add JavaDoc where possible.



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

Reply via email to