search

moloney / dcmstack

DICOM to Nifti conversion with meta data preservation
Other
72 stars 52 forks source link

voxel_order bug? #5

Closed bpinsard closed 11 years ago

bpinsard commented 11 years ago

I am using the trunk version but it seems that a recent changes have made the voxel_order option unstable. When using to_nifti_wrapper(voxel_order='RPI') on a DicomStack object and using to_filename('test.nii') and using afni 3dinfo on this file I get [-orient LAS] contrary to previous versions. As there are many commits on the function dealing with orientation, there might be a problem introduced here, or maybe I did something wrong.

moloney commented 11 years ago

Sorry about this, I switched to the more common usage where we refer to the positive directions for rows, columns, and slices. However I had not updated the documentation. I just made a commit to update the documentation now.

So if you specify to_nifti_wrapper(voxel_order='LAS') you should get what you want.

bpinsard commented 11 years ago

Sorry I was not aware that two conventions was used for nifti files orient, I noticed that using the reverse code does the job. Thanks