Skip to content

ENH: Allow afni.MaskTool to take multiple input files #2892

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 1 commit into from
Feb 25, 2019

Conversation

hstojic
Copy link
Contributor

@hstojic hstojic commented Feb 25, 2019

Summary

Fixes #2874. its a follow up from #2886 where PR was accidentally closed.

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

Made a change to the inputs of MaskTool class in AFNI interface. Used InputMultiObject as suggested, tested it out and now it works fine with multiple input images as well as far as I can tell.

Acknowledgment

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

@effigies effigies added this to the 1.1.9 milestone Feb 25, 2019
@effigies effigies changed the title ENH: Allow afni.MaskTool to take multiple input files, 2. attempt ENH: Allow afni.MaskTool to take multiple input files Feb 25, 2019
@codecov-io
Copy link

Codecov Report

Merging #2892 into master will increase coverage by 0.18%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2892      +/-   ##
==========================================
+ Coverage   67.49%   67.68%   +0.18%     
==========================================
  Files         343      343              
  Lines       43598    44327     +729     
  Branches     5426     5664     +238     
==========================================
+ Hits        29427    30002     +575     
- Misses      13467    13605     +138     
- Partials      704      720      +16
Flag Coverage Δ
#smoketests 50.48% <ø> (+0.01%) ⬆️
#unittests 65.17% <100%> (+0.25%) ⬆️
Impacted Files Coverage Δ
nipype/interfaces/afni/utils.py 81.89% <100%> (ø) ⬆️
nipype/interfaces/io.py 55.07% <0%> (ø) ⬆️
nipype/interfaces/freesurfer/preprocess.py 66.11% <0%> (ø) ⬆️
nipype/interfaces/dynamic_slicer.py 17.47% <0%> (ø) ⬆️
nipype/interfaces/nipy/preprocess.py 45.79% <0%> (ø) ⬆️
nipype/algorithms/confounds.py 67.64% <0%> (+1.29%) ⬆️
nipype/interfaces/base/core.py 88.68% <0%> (+1.79%) ⬆️
nipype/interfaces/ants/segmentation.py 76.82% <0%> (+3.09%) ⬆️
nipype/interfaces/spm/model.py 47.3% <0%> (+4.75%) ⬆️
nipype/pipeline/plugins/legacymultiproc.py 67% <0%> (+5.5%) ⬆️

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 a3f5115...7d4af09. Read the comment docs.

@effigies effigies merged commit 6071c94 into nipy:master Feb 25, 2019
yarikoptic added a commit to yarikoptic/nipype that referenced this pull request Mar 15, 2019
1.1.9 (February 25, 2019)

Full changelog: https://github.com/nipy/nipype/milestone/30?closed=1

  * FIX: Make positional arguments to LaplacianThickness require previous argument (nipy#2848)
  * FIX: Import math and csv modules for bids_gen_info (nipy#2881)
  * FIX: Ensure outputs can be listed in camino.ProcStreamlines by defining instance variable (nipy#2739)
  * ENH: Allow afni.MaskTool to take multiple input files (nipy#2892)
  * ENH: Add flags dictionary input to spm.Level1Design (nipy#2861)
  * ENH: Threshold stddev once only in TSNR (nipy#2883)
  * ENH: Add workbench.CiftiSmooth interface (nipy#2871)
  * DOC: Replace initialism typo in comment with intended phrase (nipy#2875)
  * DOC: Fix typos in ANTs Registration input documentation (nipy#2869)

* tag '1.1.9': (34 commits)
  MNT: Update changelog
  MNT: Add Katherine Bottenhorn, Paul Mihai to Zenodo
  MNT: Add kchawla-pi to Zenodo, update mailmap and ordering
  add to zenodo
  MNT: Update zenodo ordering
  Update .zenodo.json
  afni utils.py - masktool - InputMultiPath for in_file argument
  MNT: Update .zenodo ordering
  MNT: Add Oliver Contier name to .zenodo.json
  Update nipype/interfaces/spm/model.py
  ENH: Add zenodo updating script
  MNT: Update mailmap to avoid renames in script
  MNT: Update .mailmap, .zenodo.json
  MNT: Version 1.1.9
  DOC: 1.1.9 changelog
  ENH: minor - compute non degenerate stddev map once
  BF: regenerated test_auto_LaplacianThickness using wonderfully long running tools/checkspecs.py
  TEST: Thorough test of LaplacianThickness requirement cascade
  FIX: Requires error text was backwards
  import math and csv modules for bids_gen_info
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AFNI's 3dmask_tool does not accept multiple mask files
3 participants