search

SIMEXP / Data2Bids

Reorganize fmri files to make them BIDS compliant.
MIT License
10 stars 2 forks source link

nibabel minc conversion header problem #10

Open ltetrel opened 5 years ago

ltetrel commented 5 years ago

see https://github.com/nipy/nibabel/issues/712#issue-398379095

ltetrel commented 5 years ago

From Chris Markiewicz Hi @ltetrel., thanks for the report. I'm not sure that's the issue, as the affine should only be setting the first three zooms. If you're familiar with the MINC formats (or at least MINC2), would you care to dig into this and submit a pull request? I suspect the issue is here:

https://github.com/nipy/nibabel/blob/5ab94146d569dc905bfbbb5b7a4ae4b190bbd133/nibabel/minc1.py#L95-L99

It's not clear to me that we're looking for TR at all. We'll need to find how that's represented in each format, and possibly use two different approaches in MINC1 and MINC2 to return the TR.