search

neurodata / hyppo

Python package for multivariate hypothesis testing
https://hyppo.neurodata.io/
Other
215 stars 90 forks source link

Tests for Ticket #185 #406

Closed AmeliaR04 closed 9 months ago

AmeliaR04 commented 9 months ago

I enhanced the unit test suite of the hyppo package. The test suite is designed to validate the performance of the hyppo package's independence test against a reference test. This included running two simulations, a sinusoidal simulation, a clear example of a nonlinear relationship, and a null simulation as the base case. For the reference test, I choose to use scipy since itโ€™s another Python package with the functionality to measure the association between two variables. Specifically, scipy.stats has the pearsonr function which calculates the Pearson correlation coefficient, and the spearmanr function which calculates Spearman rank-order correlation coefficient.

Reference issue

185

Type of change

Tests

What does this implement/fix?

Validate the performance of independence test against a reference test.

netlify[bot] commented 9 months ago

Deploy Preview for hyppo ready!

Name Link
Latest commit 4b4e06f3da86d7a7c937e41e55e9c5efe6b33d1b
Latest deploy log https://app.netlify.com/sites/hyppo/deploys/651a08512ebc1f00080256f5
Deploy Preview https://deploy-preview-406--hyppo.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

codecov[bot] commented 9 months ago

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (9e7fc39) 96.71% compared to head (4b4e06f) 96.66%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #406 +/- ## ========================================== - Coverage 96.71% 96.66% -0.05% ========================================== Files 86 87 +1 Lines 3891 3931 +40 ========================================== + Hits 3763 3800 +37 - Misses 128 131 +3 ``` | [Files](https://app.codecov.io/gh/neurodata/hyppo/pull/406?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata) | Coverage ฮ” | | |---|---|---| | [hyppo/independence/tests/test\_additional\_sims.py](https://app.codecov.io/gh/neurodata/hyppo/pull/406?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata#diff-aHlwcG8vaW5kZXBlbmRlbmNlL3Rlc3RzL3Rlc3RfYWRkaXRpb25hbF9zaW1zLnB5) | `92.50% <92.50%> (รธ)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.