Elastix 4.8 about simpleelastix HOT 13 CLOSED

Btw, SimpleElastix deserves to be added to http://elastix.bigr.nl/wiki/index.php/Additional_tools 😉

from simpleelastix.

Yes, upgrading to 4.8 should be fairly easy. Will keep this pull request open until it is merged. Stay tuned :)

With respect to the number of threads, that option is not available because it is not exposed by the elastix API. Sorry :/

from simpleelastix.

You can now checkout the development branch to get SimpleElastix with Elastix 4.8. Would you mind running the test suite (i.e. "make test" in the SimpleITK-build directory)?

from simpleelastix.

Yes!! Will do 😄

from simpleelastix.

While compiling, I get the following error: (EDIT: Not anymore 😊, you (as in, me) shouldn't use spaces in paths!)

[ 66%] Built target SimpleITKCommon
Scanning dependencies of target SimpleElastix
Scanning dependencies of target SimpleTransformix
[ 66%] Building CXX object Code/Elastix/src/CMakeFiles/SimpleTransformix.dir/sitkSimpleTransformix.cxx.o
[ 66%] Building CXX object Code/Elastix/src/CMakeFiles/SimpleElastix.dir/sitkSimpleElastix.cxx.o
In file included from /Users/Jorrit/Development/SELX 4.8/Source/Code/Elastix/src/sitkSimpleTransformix.cxx:4:
/Users/Jorrit/Development/SELX 4.8/Source/Code/Elastix/include/sitkSimpleTransformix.h:10:10: fatal error: 'transformixlib.h' file not found
#include "transformixlib.h"
         ^
In file included from /Users/Jorrit/Development/SELX 4.8/Source/Code/Elastix/src/sitkSimpleElastix.cxx:4:
/Users/Jorrit/Development/SELX 4.8/Source/Code/Elastix/include/sitkSimpleElastix.h:10:10: fatal error: 'elastixlib.h' file not found
#include "elastixlib.h"
         ^
1 error generated.
1 error generated.
make[5]: *** [Code/Elastix/src/CMakeFiles/SimpleTransformix.dir/sitkSimpleTransformix.cxx.o] Error 1
make[4]: *** [Code/Elastix/src/CMakeFiles/SimpleTransformix.dir/all] Error 2
make[4]: *** Waiting for unfinished jobs....
make[5]: *** [Code/Elastix/src/CMakeFiles/SimpleElastix.dir/sitkSimpleElastix.cxx.o] Error 1
make[4]: *** [Code/Elastix/src/CMakeFiles/SimpleElastix.dir/all] Error 2
make[3]: *** [all] Error 2
make[2]: *** [SimpleITK-prefix/src/SimpleITK-stamp/SimpleITK-build] Error 2
make[1]: *** [CMakeFiles/SimpleITK.dir/all] Error 2
make: *** [all] Error 2

from simpleelastix.

Hi Jorrit,

Did you compiled ElastiX yourself or do you use the SuperBuild ?

If you use a system version of ElastiX (compile it yourself and USE_SYSTEM_ELASTIX) be sure to use the version from Kasper’s repositoryhttps://github.com/kaspermarstal/elastix/tree/master
The one from “mstaring/elastix” won’t work for building SimpleElastiX.

Fabien

from simpleelastix.

I use the SuperBuild, but do use the USE_SYSTEM_SWIG flag. Is there any other information I can provide?

I am compiling on OS X 10.10.5.

from simpleelastix.

Unfortunately I am unable to reproduce the issue on OS X 10.11. It may be worth starting over with a fresh clone and build directory and use "SELX4.8" instead of "SELX 4.8" to see if the space in the path is causing the issue. If this doesn't work, can you post the contents of your SimpleElastixBuild/elastix-build/src/UseElastix.cmake file?

from simpleelastix.

Very good point! Going to try it directly 😋. (Btw, I always start with a fresh clone 😊).
And good to know it's working on El Capitan!

from simpleelastix.

Yes!! It compiles! I'll let it run for the night and run make test in the morning 😊.

from simpleelastix.

100% tests passed, 0 tests failed 🎉

from simpleelastix.

🤘

from simpleelastix.

Thanks for testing!

from simpleelastix.