netneurolab / netneurotools

Useful tools from the Network Neuroscience Lab
https://netneurolab.github.io/netneurotools
BSD 3-Clause "New" or "Revised" License
57 stars 33 forks source link

Networks #106

Closed justinehansen closed 1 year ago

codecov-commenter commented 3 years ago

Codecov Report

Merging #106 (79a5877) into master (fbdf9a3) will decrease coverage by 7.48%. The diff coverage is 1.35%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   72.00%   64.52%   -7.49%     
==========================================
  Files          26       27       +1     
  Lines        1897     1990      +93     
==========================================
- Hits         1366     1284      -82     
- Misses        531      706     +175     
Impacted Files Coverage Δ
netneurotools/networks.py 6.01% <1.35%> (-3.17%) :arrow_down:
netneurotools/plotting.py 32.84% <0.00%> (-32.48%) :arrow_down:
netneurotools/tests/test_plotting.py 81.08% <0.00%> (-18.92%) :arrow_down:
netneurotools/modularity.py 59.55% <0.00%> (-4.50%) :arrow_down:
netneurotools/utils.py 82.69% <0.00%> (-3.85%) :arrow_down:
netneurotools/colors.py 92.30% <0.00%> (ø)
netneurotools/stats.py 64.31% <0.00%> (+0.42%) :arrow_up:
netneurotools/freesurfer.py 56.25% <0.00%> (+0.64%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

rmarkello commented 3 years ago

Heyo @justinehansen ! I think you accidentally included your code from #105 in this PR. If you could remove your changes to netneurotools/stats.py here to keep the PRs separate that would be awesome :raised_hands:

Let me know if you have any questions about the best way to do that :smile:

liuzhenqi77 commented 1 year ago

Hey @justinehansen , wondering was this adapted from Rick's matlab code? Should we maybe rename it match_length_degree_distribution before merging 😂?