Skip to content

BIDS MRI sidecar reader (LORIS BIDS reader PR 2) #1364

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.

Sign up for GitHub

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

Jump to bottom
Open
MaximeBICMTL wants to merge 2 commits into
base: bids_staging_branch
Choose a base branch
from
Open

BIDS MRI sidecar reader (LORIS BIDS reader PR 2) #1364

MaximeBICMTL wants to merge 2 commits into from

Conversation

@MaximeBICMTL
Copy link
Contributor

@MaximeBICMTL MaximeBICMTL commented Jan 16, 2026 •
edited
Loading

Builds on #1363 (diff)

Description

This PR adds a BidsMriSidecarJsonFile class and adopts it in the DICOM to BIDS and MRI BIDS importer pipeline. The code is rather trivial.

@github-actions github-actions bot added the Language: Python Issue or PR related to the Python codebase label Jan 16, 2026
@MaximeBICMTL MaximeBICMTL force-pushed the loris_bids_reader_2_mri_sidecar branch from ba750fb to aded181 Compare January 16, 2026 11:28
@MaximeBICMTL MaximeBICMTL mentioned this pull request Jan 16, 2026
Open
@MaximeBICMTL MaximeBICMTL changed the base branch from main to bids_staging_branch January 16, 2026 11:31
@MaximeBICMTL MaximeBICMTL added Complexity: Simple Issue or PR that should be simple to implement, review, or test Area: BIDS Issue or PR related to the handling of BIDS datasets Pipeline: BIDS importer PR or issue related to the BIDS importer Pipeline: DICOM to BIDS PR or issue related to the DICOM to BIDS converter Category: Refactor Issue or PR that aims to improve the existing code and removed Area: BIDS Issue or PR related to the handling of BIDS datasets labels Jan 16, 2026
@MaximeBICMTL MaximeBICMTL marked this pull request as ready for review January 16, 2026 11:50
@MaximeBICMTL MaximeBICMTL requested a review from cmadjar January 16, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cmadjar cmadjar Awaiting requested review from cmadjar

Assignees

No one assigned

Labels

Category: Refactor Issue or PR that aims to improve the existing code Complexity: Simple Issue or PR that should be simple to implement, review, or test Language: Python Issue or PR related to the Python codebase Pipeline: BIDS importer PR or issue related to the BIDS importer Pipeline: DICOM to BIDS PR or issue related to the DICOM to BIDS converter

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MaximeBICMTL