rordenlab/dcm2niix

.par PhaseEncodingDirection

fliem opened this issue · 3 comments

fliem commented

I converted .par files with different PhaseEncodingDirection and saw that both BIDS sidecar files show "PhaseEncodingDirection": "i"
To my knowledge, .par files don't list PhaseEncodingDirection. Could you please remove the PhaseEncodingDirection entry from the .json file for conversion from .par files. Thanks

neurolabusc commented

Well spotted - issue was specific to the BIDS header (not the notes field of the NIfTI header). Please checked the updated code, which should omit PhaseEncodingDirection if unknown.

jonclayden commented

I think there may be a problem with the fix, because the new conditional statement inserted matches the one that it should be an alternative to (two lines earlier). Presumably one should test against 'R' and one against 'C'.

neurolabusc commented

Fixed. Also, effective echo spacing in BIDS now correctly is omitted if we can not calculate it due to unknown phase-encoding.