rasbt / mlxtend

A library of extension and helper modules for Python's data analysis and machine learning libraries.
https://rasbt.github.io/mlxtend/
Other
4.82k stars 853 forks source link

Possible mistake in paired_ttest_resampled doc #975

Closed Hardwarize closed 1 year ago

Hardwarize commented 1 year ago

Describe the documentation issue

I think that there is an error in the T-value formula in the Resampled paired t test doc page.

image

It seems that, in the denominator, p is raised to (i) - p

Suggest a potential improvement or addition

I think that this: $$t = \frac{\overline{p} \sqrt{k}}{\sqrt{\sum_{i=1}^{k}(p^{(i) - \overline{p}})^2 / (k-1)}}.$$

should be replace by this: $$t = \frac{\overline{p} \sqrt{k}}{\sqrt{\sum_{i=1}^{k}(p^{(i)} - \overline{p})^2 / (k-1)}}.$$

Thanks for this amazing library!

rasbt commented 1 year ago

Thanks a lot for the note! You are right, that was a formatting issue. Just fixed it via #976