Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add afq to stats #1768

Merged
merged 4 commits into from Mar 25, 2019

Conversation

@arokem
Copy link
Member

commented Mar 13, 2019

Following #1765 (comment)

@pep8speaks

This comment has been minimized.

Copy link

commented Mar 13, 2019

Hello @arokem, Thank you for updating !

Line 127:1: E402 module level import not at top of file

Comment last updated at 2019-03-13 21:10:28 UTC
arokem added 2 commits Mar 13, 2019
@codecov-io

This comment has been minimized.

Copy link

commented Mar 14, 2019

Codecov Report

❗️ No coverage uploaded for pull request base (master@78f74e8). Click here to learn what that means.
The diff coverage is 94.93%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1768   +/-   ##
=========================================
  Coverage          ?   83.88%           
=========================================
  Files             ?      120           
  Lines             ?    14570           
  Branches          ?     2295           
=========================================
  Hits              ?    12222           
  Misses            ?     1824           
  Partials          ?      524
Impacted Files Coverage Δ
dipy/segment/bundles.py 91.47% <ø> (ø)
dipy/stats/analysis.py 75% <94.93%> (ø)
@skoudoro

This comment has been minimized.

Copy link
Member

commented Mar 25, 2019

This looks good to me. Thank you @arokem for the refactoring. merging and fix #1765

@skoudoro skoudoro merged commit 193a8d9 into nipy:master Mar 25, 2019

4 checks passed

codecov/patch No report found to compare against
Details
codecov/project No report found to compare against
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.