Skip to content

fix: collect data interactions with sessions and bids filters - #1063

Merged
effigies merged 2 commits into
masterfrom
fix/collect-data-filtering
Sep 28, 2026
Merged

effigies merged 2 commits into
masterfrom
fix/collect-data-filtering

Conversation

@mgxd

@mgxd mgxd commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

motivated by nipreps/fmriprep#3607

a filter selecting a subset of session_id like:

session_id=['anat', 'func1']
bids_filters={'bold': {'session': 'func1'}})

raised Conflicting entities for "session". Instead, filters should allow narrowing from these values, or when session_id=None. However, the case:

session_id=['anat', 'func1']
bids_filters={'bold': {'session': 'func2'}})

should still raise.

@codecov

codecov Bot commented Sep 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.13%. Comparing base (c2d86ef) to head (90c2f32).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1063      +/-   ##
==========================================
- Coverage   67.15%   67.13%   -0.02%     
==========================================
  Files          54       54              
  Lines        6850     6847       -3     
  Branches      852      850       -2     
==========================================
- Hits         4600     4597       -3     
  Misses       2037     2037              
  Partials      213      213              

☔ View full report in Codecov by Harness.
📢 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.

@effigies effigies left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Does this need backporting into 1.14.x for the LTS?

@effigies
effigies merged commit 2de2618 into master Sep 28, 2026
15 checks passed
@effigies
effigies deleted the fix/collect-data-filtering branch September 28, 2026 18:38
effigies pushed a commit that referenced this pull request Sep 28, 2026
* fix: allow bids_filters to narrow sessions in collect_data

* tst: verify session / filter behavior
mgxd added a commit to nipreps/fmriprep that referenced this pull request Sep 30, 2026
25.2.6 (September 29, 2026)

Bug-fix release in the 25.2.x series.

This release updates downstream dependencies to fix how session selection
interacts with ``--bids-filter-file``.

  * DOC: Add note on StartTime metadata (#3670)
  * MNT: Bump niworkflows to 1.15.2 (nipreps/niworkflows#1063)
  * MNT: Bump sdcflows to 2.15.1 (nipreps/sdcflows#559)
  * MNT: Fix py310-min tests on CI (#3671)
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.

2 participants