[SyneRBI/SIRF-SuperBuild] bump CIL deps (Issue #993)
Casper da Costa-Luis
casperdcl created an issue (SyneRBI/SIRF-SuperBuild#993)- CIL version TomographicImaging/CIL#2218
- External_TomoPhantom.cmake
- External_astra-python-wrapper.cmake
- External_astra-toolbox.cmake
—
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#993)If the long term plan is to be able to install SIRF with conda, we'd be better removing TomoPhantom from the SuperBuild and add it as dependency line in the requirements.yml.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
KrisThielemans left a comment (SyneRBI/SIRF-SuperBuild#993)I don't know what TomoPhantom depends on, but I'm entirely ok to no longer add it to the default list of things to build, i.e. not add it add https://github.com/SyneRBI/SIRF-SuperBuild/blob/f9a9426f24594878442a0f7517619f1ee3febdcb/SuperBuild.cmake#L313-L317
You could of course keep External_TomoPhantom.cmake if you think it's useful (but I suppose it'd have to be tested to remain useful). Same for ASTRA really, or indeed CIL.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.