.par PhaseEncodingDirection

Question

.par PhaseEncodingDirection

fliem opened this issue 8 years ago · comments

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

Chris Rorden · Answer 1 · Mon Jan 09 2017 23:42:28 GMT+0800 (China Standard Time)

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.

Jon Clayden · Answer 2 · Thu Jan 12 2017 22:50:34 GMT+0800 (China Standard Time)

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'.

Chris Rorden · Answer 3 · Fri Jan 13 2017 00:12:59 GMT+0800 (China Standard Time)

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