[prev in list] [next in list] [prev in thread] [next in thread]
List: jakarta-commons-dev
Subject: [jira] [Commented] (MATH-1502) KolmogorovSmirnovTest#exactP value is incorrect in 3.6.1 potential bu
From: "Gilles Sadowski (Jira)" <jira () apache ! org>
Date: 2019-10-30 23:49:00
Message-ID: JIRA.13265092.1572369680000.68114.1572479340217 () Atlassian ! JIRA
[Download RAW message or body]
[ https://issues.apache.org/jira/browse/MATH-1502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16963521#comment-16963521 \
]
Gilles Sadowski commented on MATH-1502:
---------------------------------------
bq. I'm not too familiar with the KolmogorovSmirnovTest
I'm not either.
Hopefully, the set of existing tests ensure correct behaviour. But additional test \
are welcome (to be added to the development version towards the upcoming version \
4.0).
> KolmogorovSmirnovTest#exactP value is incorrect in 3.6.1 potential bug
> ----------------------------------------------------------------------
>
> Key: MATH-1502
> URL: https://issues.apache.org/jira/browse/MATH-1502
> Project: Commons Math
> Issue Type: Bug
> Affects Versions: 3.6.1
> Reporter: Ethan Mahintorabi
> Priority: Critical
> Attachments: exact_vs_approximate.png
>
>
> I believe that I have found a bug in the calculation of the exactP value in \
> KolmogorovSmirnovTest.java in version 3.6.1.
> In version 3.4.1 KolmogorovSmirnovTest#exactP(0.5633587786259542, 3, 3, true) = 0.6
>
> Where as
>
> In version 3.6.1 KolmogorovSmirnovTest#exactP(0.5633587786259542, 3, 3, true) = \
> 0.09999999999999998
>
> The approximateP value has stayed the same at ~=0.72. This seems to correspond with \
> the change in methods of calculation in e38bbb9f4191d0d21dea0ba31fdc131b97a5597b
> [https://github.com/apache/commons-math/commit/e38bbb9f4191d0d21dea0ba31fdc131b97a5597b#diff-34100572fcc8e6537bc7445a0e82b900]
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic