Conversation

salma1601

Summary

Fixes #3140 .

List of changes proposed in this PR (pull-request)

added name_template, name_source and kepp_extension to out_matrix

Acknowledgment

  • (Mandatory) I acknowledge that this contribution will be available under the Apache 2 license.

@satra

@salma1601 - one of the doctests needs to be updated with this: -1Dmatrix_save functional_allineated.aff12

@codecov

Codecov Report

Merging #3141 into master will decrease coverage by 2.75%.
The diff coverage is 72.97%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3141      +/-   ##
==========================================
- Coverage   67.62%   64.86%   -2.76%     
==========================================
  Files         299      299              
  Lines       39480    39493      +13     
  Branches     5220     5219       -1     
==========================================
- Hits        26698    25618    -1080     
- Misses      12070    12826     +756     
- Partials      712     1049     +337
FlagCoverage Δ
#smoketests?
#unittests64.86% <72.97%> (-0.03%)⬇️
Impacted FilesCoverage Δ
nipype/pipeline/plugins/tools.py78.94% <ø> (-1.32%)⬇️
nipype/interfaces/base/core.py83.85% <0%> (-3.12%)⬇️
nipype/pipeline/plugins/sgegraph.py14.94% <0%> (ø)⬆️
nipype/pipeline/engine/nodes.py78.15% <100%> (-5.16%)⬇️
nipype/pipeline/plugins/multiproc.py78.78% <100%> (-2.47%)⬇️
nipype/info.py92.3% <100%> (-1.54%)⬇️
nipype/interfaces/fsl/model.py55.14% <100%> (-25.44%)⬇️
nipype/pipeline/plugins/legacymultiproc.py68.81% <100%> (+5.99%)⬆️
nipype/__init__.py34.37% <33.33%> (-24.36%)⬇️
nipype/interfaces/io.py56.41% <42.85%> (-3.27%)⬇️
... and 40 more

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 5d68df6...85aeffc. Read the comment docs.

@effigies

@salma1601 You might have an old version of black installed. I merged master and reran make specs for you. Is this ready to go otherwise?

@effigieseffigies added this to the 1.5.0 milestone Feb 24, 2020
@effigieseffigies modified the milestones: 1.5.0, 1.5.1 Jun 3, 2020
@effigies

Just checking back in on this in the run-up to another release. Apologies for the short notice...

@effigieseffigies modified the milestones: 1.5.1, future Aug 16, 2020
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

afni Allineate caching fails when out_matrix is specified