.par PhaseEncodingDirection

[fliem]

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]

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]

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]

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