WIN64 compatible problem about dcm2niix HOT 5 CLOSED

Hmm. I usually compile this with Clang/LLVM (OS X), GCC (Linux) or GCC+cygwin (Windows). I did update the code so it would compile with Visual Studio, but I did not test it. I do not currently have a system with Visual Studio installed. My first question is whether you need the 64-bit version. If you do work out the root cause I would be happy to know, but at the moment it is not high on my priority list to fix as it does not seem like it will influence most users.

from dcm2niix.

I temporarily solved the problem by increasing the HEAP and STACK sizes within Visual Studio . I personally think the problem is we have too many variables and static array within data structures, therefore require larger size with struct alignment for WIN64. And http://www.codeguru.com/cpp/v-s/debug/article.php/c19241/Adventures-with-chkstk.htm have a good explanation.

from dcm2niix.

That makes perfect sense. Can you try the new version I uploaded to see if this fixes the problem without having to change the Visual Studio defaults? I changed the lines below - the intention is to allow the software to convert all the data in a single pass, so I assume a huge number of images.

#if UINTPTR_MAX == 0xffffffff
    nameList.maxItems = 68000; // 32-bit larger requires more memory, smaller more passes 
#elif UINTPTR_MAX == 0xffffffffffffffff
    nameList.maxItems = 34000; // 64-bit larger requires more memory, smaller more passes 
#endif

from dcm2niix.

Hello-
Please try the latest (2June2015) release. This dramatically reduces memory usage. The challenge is supporting simultaneously (1) thousands of simple 2D DICOMs [e.g. Siemens DTI] and (2) very complicated 4D DICOMs [e.g. Philips DTI]. The new version only batches the 2D images and immediately converts a 4D image, so it never needs to store 4D information long term.

from dcm2niix.

Thanks to you contribution, the improvement definitely solved my
problem with visual studio under WIN64, and it is more efficient.

from dcm2niix.