Closed rmarkello closed 3 years ago
Merging #91 (caca281) into master (c0b4ce8) will decrease coverage by
0.31%
. The diff coverage is52.63%
.
@@ Coverage Diff @@
## master #91 +/- ##
==========================================
- Coverage 64.59% 64.27% -0.32%
==========================================
Files 23 22 -1
Lines 1590 1601 +11
==========================================
+ Hits 1027 1029 +2
- Misses 563 572 +9
Impacted Files | Coverage Δ | |
---|---|---|
netneurotools/__init__.py | 100.00% <ø> (ø) |
|
netneurotools/modularity.py | 59.55% <ø> (ø) |
|
netneurotools/plotting.py | 6.57% <0.00%> (-0.06%) |
:arrow_down: |
netneurotools/stats.py | 78.39% <58.82%> (-2.26%) |
:arrow_down: |
netneurotools/tests/test_stats.py | 100.00% <0.00%> (ø) |
|
netneurotools/tests/test_utils.py | 100.00% <0.00%> (ø) |
|
netneurotools/tests/test_cluster.py | 100.00% <0.00%> (ø) |
|
netneurotools/tests/test_freesurfer.py | 100.00% <0.00%> (ø) |
|
netneurotools/tests/test_datasets.py | 98.71% <0.00%> (+0.12%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 38add68...bc63e79. Read the comment docs.
Adds functionality to
netneurotools.stats.efficient_pearsonr()
such that it can handle NaN values in either (or both) of the inputs. Thenan_policy
parameter is adapted fromscipy.stats
mechanics; when set to'omit'
it will ignore the NaN values in both arrays and compute the correlation with the relevant entries excluded.To Do: