Conditional Correlation Tests
Reference issue
Closes #332
Type of change
Addition of three correlation based conditional independence tests.
- Partial pearson correlation test (univariate only)
- Partial distance correlation (Szekely 2014)
- Conditional distance correlation (Wang 2015)
Others
- Simulations for conditional independence
- Power curves for partial dcorr and conditional dcorr
- Tests
Additional information
Few functions in tools were changed to accommodate for three input data matrices.
-
compute_dist -
_check_distmat -
_perm_stat -
perm_test
Deploy Preview for hyppo ready!
| Name | Link |
|---|---|
| Latest commit | 89910e27909f1831ee4885fe4abc220a9f3fb541 |
| Latest deploy log | https://app.netlify.com/sites/hyppo/deploys/66df0968937455000937001e |
| Deploy Preview | https://deploy-preview-349--hyppo.netlify.app |
| Preview on mobile | Toggle 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 Report
Attention: Patch coverage is 97.28000% with 17 lines in your changes missing coverage. Please review.
Project coverage is 96.79%. Comparing base (
8f12a27) to head (89910e2). Report is 1 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #349 +/- ##
==========================================
+ Coverage 96.70% 96.79% +0.09%
==========================================
Files 86 96 +10
Lines 3879 4490 +611
==========================================
+ Hits 3751 4346 +595
- Misses 128 144 +16
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@j1c The only thing missing is a few unit tests for some of the error checks. Once those are added, I'll merge
@j1c Let me know when you believe that everything looks good and I'll review
@j1c Let me know when you believe that everything looks good and I'll review
This is good to go.