ENH: Toggle requirement for any image type BIDSDataGrabber, add task filtering when collecting PET data

[mnoergaard]

Needed to update this part to be able to run PETPrep without the workflow looking for functional data, when not using the --anat-only flag.

[codecov]

Codecov Report

Attention: Patch coverage is 97.05882% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.53%. Comparing base (37a8147) to head (9fd80dc).
Report is 15 commits behind head on master.

Files with missing lines	Patch %	Lines
niworkflows/utils/bids.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #951      +/-   ##
==========================================
+ Coverage   71.42%   71.53%   +0.10%     
==========================================
  Files          89       89              
  Lines        8631     8648      +17     
  Branches     1075     1075              
==========================================
+ Hits         6165     6186      +21     
+ Misses       2232     2230       -2     
+ Partials      234      232       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[mgxd]

This breaks compatibility with other tools depending on these - rather than this, we should add deviations where necessary.

[mnoergaard]

Many thanks to both of you for your feedback. These recent changes seem to work on the petprep side - WDYT?

[mgxd]

@mnoergaard I cleaned up the BIDSDataGrabber logic in 42cb1a2 - mainly generalizing require_* to encompass all the image types we support. A few changes of note:

• renamed require_funcs to require_bold, which is more in-line with the query and the other types.
• require_pet will not implicitly disable require_bold - they should be controlled separately

updated

[mgxd]

Looks good to me, but I'll let you test the petprep integration prior to merging 😃

[mnoergaard]

Awesome - many thanks, @mgxd ! I will go ahead and test it and get back to you. Currently running things with the updated version from 4 hours ago. So good so far!