ENH: More flexible BIDSDataGrabber creation

[effigies]

Changes proposed in this pull request

• Remove outfields parameter - redundant with output_query input
• Allows inputs to be specified in kwargs
• auto tests

[adelavega]

Cool. Although I think I had the outfields parameter because that gets called on init whereas output_query input is not called until run time. I figured the former would allow checking a workflow has the correct inputs/ouputs, but maybe that's not necessary?

[adelavega]

Any desire to take care #2207 while you're at it? Although I'm not sure it was confirmed whether or not Python 2 support will be entirely dropped out of pybids, so maybe just add a skiptest if pybids was not installed in editable mode?

[oesteban]

@adelavega, Chris' solution is equivalent to having outfields. Inputs can be passed through __init__ so you could always have done:

bids = BIDSDataGrabber(output_query={ ... })

and that would have the same effect as:

bids = BIDSDataGrabber(outfields={ ... })
bids.inputs.output_query = { ... }

[codecov-io]

Codecov Report

Merging #2228 into master will increase coverage by <.01%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master    #2228      +/-   ##
==========================================
+ Coverage   72.31%   72.32%   +<.01%     
==========================================
  Files        1181     1181              
  Lines       58939    58893      -46     
  Branches     8488     8476      -12     
==========================================
- Hits        42622    42593      -29     
+ Misses      14934    14920      -14     
+ Partials     1383     1380       -3

Flag	Coverage Δ
#smoketests	72.32% <71.42%> (ø)	⬆️
#unittests	69.88% <71.42%> (ø)	⬆️

Impacted Files	Coverage Δ
nipype/interfaces/bids_utils.py	81.03% <57.14%> (-0.22%)	⬇️
...pype/interfaces/tests/test_auto_BIDSDataGrabber.py	78.57% <78.57%> (ø)
nipype/interfaces/ants/utils.py	85.71% <0%> (-1.42%)	⬇️
...pe/interfaces/ants/tests/test_auto_Registration.py	85.71% <0%> (ø)	⬆️
nipype/utils/config.py	67.64% <0%> (ø)	⬆️
...aces/ants/tests/test_auto_ComposeMultiTransform.py
nipype/interfaces/ants/registration.py	75% <0%> (+2.13%)	⬆️

---

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 0af4d56...425a570. Read the comment docs.