[jira] [Commented] (OPENNLP-845) Evaluator.evaluateSample() sends the wrong argument to listener.correctlyClassified()

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

GitHub user smarthi opened a pull request:

    https://github.com/apache/opennlp/pull/22

    OPENNLP-845: Evaluator.evaluateSample() sends the wrong argument to listener.correctlyClassified()

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/smarthi/opennlp OPENNLP-845

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/opennlp/pull/22.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #22
    
----
commit 444fa2bc722d92a3b74b156bb01fed81ca92736f
Author: smarthi <sm...@apache.org>
Date:   2016-12-25T22:49:41Z

    OPENNLP-845: Evaluator.evaluateSample() sends the wrong argument to listener.correctlyClassified()

----


> Evaluator.evaluateSample() sends the wrong argument to listener.correctlyClassified()
> -------------------------------------------------------------------------------------
>
>                 Key: OPENNLP-845
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-845
>             Project: OpenNLP
>          Issue Type: Bug
>            Reporter: Dave Kincaid
>
> When the evalation of a sample The implementation of Evaluator.evaluateSample() puts the predicted document into the first argument in the call to listener.correctlyClassified instead of the sample being evaluated. This causes any extra information in the document sample to be lost.
> The call should be listener.correctlyClassified(sample, predicted)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)