-
Notifications
You must be signed in to change notification settings - Fork 532
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
Fix catmatvec #2527
Fix catmatvec #2527
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2527 +/- ##
=======================================
Coverage 66.88% 66.88%
=======================================
Files 327 327
Lines 42490 42490
Branches 5269 5269
=======================================
Hits 28421 28421
Misses 13367 13367
Partials 702 702
Continue to review full report at Codecov.
|
@salma1601 have you tried this without adding the |
yes and you are right, it works without it ! removed |
nipype/interfaces/afni/utils.py
Outdated
@@ -616,7 +619,6 @@ def _format_arg(self, name, spec, value): | |||
for i in value])) | |||
return super(CatMatvec, self)._format_arg(name, spec, value) | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you re-add this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes sorry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #2526 .
Changes proposed in this pull request
name_template
andname_source
for generating the out_file_list_outputs
method