Skip to content

DVARS fix & improvements #1827

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

Merged
merged 12 commits into from
Feb 23, 2017
Merged

DVARS fix & improvements #1827

merged 12 commits into from
Feb 23, 2017

Conversation

chrisgorgo
Copy link
Member

Fixes #1821.

@satra
Copy link
Member

satra commented Feb 22, 2017

@chrisfilo - needs remerge

warnings.filterwarnings('error')

# voxelwise standardization
diff_vx_stdz = np.square(func_diff) / np.array([diff_sdhat] * func_diff.shape[-1]).T
diff_vx_stdz = np.square(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

**2 would be more readable to me than np.square, but it is just cosmetic. Maybe you prefer np.square :)

@codecov-io
Copy link

Codecov Report

Merging #1827 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1827      +/-   ##
==========================================
+ Coverage   72.71%   72.71%   +<.01%     
==========================================
  Files        1059     1059              
  Lines       52550    52556       +6     
==========================================
+ Hits        38211    38217       +6     
  Misses      14339    14339
Flag Coverage Δ
#unittests 72.71% <100%> (ø)
Impacted Files Coverage Δ
nipype/algorithms/tests/test_confounds.py 94.73% <100%> (+1.4%)
nipype/algorithms/confounds.py 80.23% <100%> (-0.12%)

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 a8e8d95...473d0dd. Read the comment docs.

@chrisgorgo chrisgorgo merged commit 0ac8143 into nipy:master Feb 23, 2017
@effigies effigies mentioned this pull request Mar 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants