Open sprague252 opened 1 year ago
Patch coverage: 100.00
% and project coverage change: +3.70
:tada:
Comparison is base (
e03407a
) 90.04% compared to head (7bba907
) 93.75%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Resolves #266
This change adds a new keyword parameter
ddof
toPowerDivergenceTest
andChisqTest
allowing users to change the number of degrees of freedom used for calculating p-values. This works the same as theddof
parameter in scipy.stats.chisquare. See the documentation for that function athttps://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.chisquare.html.
The
ddof
value defaults to 0 (no changes to the degrees of freedom). A supplied value forddof
decreases the number of degrees of freedom by the value ofddof
. This is important when the distribution probabilities are calculated using parameters from measured data (e.g., mean and sample standard deviation), reducing the number of degrees of freedom.I updated the docstrings for
PowerDivergenceTest
andChisqTest
and added a test forChisqTest
withddof=2
totest/power_divergence.jl
. (It passes.)