[SyneRBI/SIRF-SuperBuild] Error building Tomophantom (Issue #932)
NicoleJurjew
When I try to build with build CIL = ON, I get an error. This seems to be from Tomophantom.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
The error said something about cannot find msvc compiler...
I see that Tomophantom has move on dramatically now. Should we update it? Or should we even include it? Currently, it is enabled via BUILD_CIL. Might need a separate BUILD variable then.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
But we are using a specific version, despite the package changes.
https://github.com/SyneRBI/SIRF-SuperBuild/blob/5f1a7f498bf6f41551c19eb53798b5d33af43eec/version_config.cmake#L127
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
We do have a frozen version, which is good, but it is old and OS/compilers etc move on, which possibly caused this problem.
I'm not sure how easy it will be for @NicoleJurjew to reproduce this now (and how much time she has for it!). @NicoleJurjew can you just give OS, CMake and gcc version?
In any case, I think it'd be good to be able to switch-off building of TomoPhantom, independent of CIL.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
paskino left a comment (SyneRBI/SIRF-SuperBuild#932)Should we discuss whether to include TomoPhantom and CCPi-Regularisation as dependencies of CIL?
https://github.com/SyneRBI/SIRF-SuperBuild/blob/c5251d17dc2a4a612bc4b4247b07704d4ccb448b/SuperBuild.cmake#L314
Potentially we could add a BUILD_TomoPhantom etc to force the build.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
KrisThielemans left a comment (SyneRBI/SIRF-SuperBuild#932)sure. I thought you needed it for the CIL-demos, but by all means, make it optional.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
paskino left a comment (SyneRBI/SIRF-SuperBuild#932)See TomographicImaging/CIL-Demos#254
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
paskino left a comment (SyneRBI/SIRF-SuperBuild#932)Currently TomoPhantom is not a dependency anymore, so we might close this
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.