[log in to unmask]

Google Groups

Kris Thielemans <[log in to unmask]>: Nov 11 02:49PM -0800 We currently find files explicitly https://github.com/SyneRBI/SIRF/blob/b1a476342b5b7674e798c314d2ae32364e7fdde6/src/xGadgetron/cGadgetron/CMakeLists.txt#L45 Newer Gadgetron exports a CMake config ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 07:49AM -0800 @paskino pushed 1 commit.   b818fd43b41be9ec50ffe767d2a721a06c2ba8da update packages   -- View it on GitHub: ...more

Michael Hansen <[log in to unmask]>: Nov 11 07:51AM -0800 Just a heads up on this. We have a PR going through Gadgetron right now to remove the CPR dependency: https://github.com/gadgetron/gadgetron/pull/1173. That may help you here.   -- ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 07:54AM -0800 Thanks @hansenms that would solve some issues, however I just made it work...   Next problem is ``` CMake Error at apps/gadgetron/test/CMakeLists.txt:3 (add_executable): Target "server_tests" ...more

Kris Thielemans <[log in to unmask]>: Nov 11 08:15AM -0800 @KrisThielemans commented on this pull request.       > @@ -108,17 +113,19 @@ set(DEFAULT_NIFTYREG_TAG 99d584e2b8ea0bffe7e65e40c8dc818751782d92 ) set(DEFAULT_NIFTYREG_REQUIRED_VERSION 1.5.68) ...more

Kris Thielemans <[log in to unmask]>: Nov 11 08:39AM -0800 I think that this is a Gadgetron bug. I've created https://github.com/gadgetron/gadgetron/issues/1175. I believe it implies we have to build Gadgetron with `BUILD_TESTING=ON`.   -- ...more

Kris Thielemans <[log in to unmask]>: Nov 11 08:41AM -0800 @hansenms could you comment if we really should try to build Gadgetron 4.1? It is from Nov 2019. For instance, I found some `GTest` related changes that were merged since then, ...more

Michael Hansen <[log in to unmask]>: Nov 11 08:50AM -0800 @KrisThielemans if you are moving to a newer version, I would go with the latest version. It is the one that was built most recently and where dependencies are most well understood. There was a ...more

Kris Thielemans <[log in to unmask]>: Nov 11 08:53AM -0800 @paskino had already created https://github.com/gadgetron/gadgetron/issues/1174, so I've closed my issue. I now have a PR to fix Gadgetron https://github.com/gadgetron/gadgetron/pull/1176   -- ...more

Kris Thielemans <[log in to unmask]>: Nov 11 08:55AM -0800 @KrisThielemans commented on this pull request.       > @@ -6,10 +6,10 @@ apt-get update -qq apt-get install -yq --no-install-recommends \ libhdf5-serial-dev \ libboost-dev ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 10:53AM -0800 @paskino pushed 1 commit.   5892c2bc17ac03e884cd47a02a02db5978fe3456 add libboost-optional-dev   -- View it on GitHub: ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 11:00AM -0800 @paskino pushed 1 commit.   f66b998a57a26493d508609411a1b6354e9030c2 update boost dependency and build gadgetron testing   -- View it on GitHub: ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 11:45AM -0800 @paskino pushed 1 commit.   a948fc3fa5191ab69318e7b0e1f5a3714d12da9f build Gadgetron master   -- View it on GitHub: ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 12:03PM -0800 @paskino pushed 2 commits.   79ecef6baee5aa8871a55e974932594cd0ecf77a looks like we need date 36c4a021df87db8804e528d71d43296d5a3068e4 set install for date   -- View it on GitHub: ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 12:06PM -0800 @paskino pushed 1 commit.   c06065463efc1fcd2f3539d3e777f624a8fb835c fix date url/tag   -- View it on GitHub: ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 12:57PM -0800 @paskino pushed 1 commit.   7d37213ad19ffe35bf3f3b33f3df3e4cca34568d build SIRF   -- View it on GitHub: ...more

Edoardo Pasca <[log in to unmask]>: Nov 11 12:59PM -0800 @paskino pushed 1 commit.   6a4a9cdef58760343d02f2a73ecb7be75372846a build ISMRMRD master   -- View it on GitHub: ...more

Kris Thielemans <[log in to unmask]>: Nov 11 01:30PM -0800 @paskino, https://github.com/gadgetron/gadgetron/pull/1173 has now been merged< so you can remove the CPR requirement again (!). I'd leave the files there, but just remove it from the Gadgetron ...more

Kris Thielemans <[log in to unmask]>: Nov 11 02:02PM -0800 @KrisThielemans pushed 2 commits.   48b57b8c782767e37841cd969a7d3b292f097a62 [GHA] fix yaml a333c62cf343cb811f36495a7fc7e0b206a8f196 make boost libs and version consistent between files   -- ...more

Kris Thielemans <[log in to unmask]>: Nov 11 02:12PM -0800 @KrisThielemans pushed 1 commit.   7c7a264a2e52c777459d6f288347a265574aaa2a corrected boost package names   -- View it on GitHub: ...more

Kris Thielemans <[log in to unmask]>: Nov 11 02:33PM -0800 I've fixed the (normal) GHA CI and boost version stuff. Note that this means that 18.04 jobs now fall over quickly as the system boost is too old. Could change them to build boost out of curiosity ...more

Kris Thielemans <[log in to unmask]>: Nov 11 02:34PM -0800 I've cancelled all docker jobs on GHA as they don't test this PR (see #775)   -- Reply to this email directly or view it on GitHub: ...more

Kris Thielemans <[log in to unmask]>: Nov 11 02:20PM -0800 https://github.com/SyneRBI/SIRF-SuperBuild/blob/a0ad95c056cd7f666d8e09f296d1234227d50bef/docker/Dockerfile#L127-L128 and ...more

Evgueni Ovtchinnikov <[log in to unmask]>: Nov 11 04:41AM -0800 @evgueni-ovtchinnikov pushed 1 commit.   87ba87d006e2cc8c8a8911a3409288d3e0aadfbd [ci skip] restored several-chains demo following PR 1143 bugfix   -- View it on GitHub: ...more

You have received this digest because you're subscribed to updates for this group. You can change your settings on the group membership page. To unsubscribe from this group and stop receiving emails from it send an email to [log in to unmask].