Gabriel A. Devenyi
Gabriel A. Devenyi
Given two files, in both MINC and NIFTI format: GAD - scan of me T1 - convert to MINC with dcm2mnc, minc-toolkit/1.9.16 - convert to NIFTI with dcm2niix/1.0.20171215 mni_icbm152_t1_tal_nlin_asym_09c -...
Additional tests: - antsRegistration SyN to produce nonlinear - save as minc/nifti - confirm forward/reverse transformations also work as expected with antsApplyTransforms and mincresample - nlfit to produce nonlinear -...
All conversions and affine registrations now pass my listed tests. The following still don't work - nlfip -> antsApplyTransform - mincresample applying the NLIN fields produced by ANTs ITK and...
PR https://github.com/InsightSoftwareConsortium/ITK/pull/194 addresses the issue with MINC NLIN transforms only being generated one-way and ITK not properly handling the invert flag on such transforms. Still remaining is I need to...
Added patch now properly handles Grid transforms to/from MINC land. My local tests of - mincresample applying ITK generated transforms - antsApplyTransforms applying grid transforms generated by nlpfit Now work...
Thanks @dzenanz I have removed the extraneous Fills.
@thewtex I think this change is breaking the current tests, is that correct? If so, any hints as to how to I would go about sorting that out?
@thewtex that's what I'm asking? how do I go about checking that locally :)
All the test are failing with errors like this, with can't read a file. ``` /scratch/ITK-build/bin/ITKIOMINCTestDriver "--compare" "/scratch/ITK-build/ExternalData/Modules/IO/MINC/test/Input/t1_z+_ushort_trans.mnc" "/scratch/ITK-build/Testing/Temporary/t1_z+_ushort_trans.mnc" "itkMINCImageIOTest4" "/scratch/ITK-build/ExternalData/Modules/IO/MINC/test/Input/t1_z+_ushort_trans.mnc" "/scratch/ITK-build/Testing/Temporary/t1_z+_ushort_trans.mnc" "427590.7957" "-8.195997123" "72.45943721" "-3.148635961" Image:/scratch/ITK-build/ExternalData/Modules/IO/MINC/test/Input/t1_z+_ushort_trans.mnc sum=427590.7957 COM=[8.195997127, -72.45943721,...
Oops. I see it now. Its testing COM in RAS space, and that's wrong now :+1: Patch incoming