BIC-MNI / minc-tools

Basic minc-tools from former minc repository
Other
30 stars 25 forks source link

mnc2dcm tools on develop not installed #57

Open gdevenyi opened 7 years ago

gdevenyi commented 7 years ago

I thin a small tweak to cmake is needed to install these tools.

andrewjanke commented 7 years ago

This is likely because they aren't complete. They 90% work but it was more an itch of mine that I scratched a while back rather than a complete converter. Am also not sure of the licencing on the DICOM perl module, I made some modifications to it to get this to work IIRC.

gdevenyi commented 7 years ago

Okay. dcmsort is fantastic. I use it constantly.

andrewjanke commented 7 years ago

I should add that I think the best way forward for this is to port both dcmsort and mnc2dcm to use dcmtk instead of DICOM.pm. That would be dcmdump for dcmsort and img2dcm for mnc2dcm.

dcmtk has long term support unlike the DICOM perl module I modified such that it would write DICOM in addition to reading it. The main issue was that the DICOM perl module doesn't create correct DICOM UID's IIRC. img2dcm does. The main issue would be to check that dcmsort will still run with acceptable speed.

If this is done then it would make sense to include it as part of minc-tools proper via cmake.