============================================================================ Today's topic summary ============================================================================ Group: [log in to unmask] URL: https://groups.google.com/forum/?utm_source=digest&utm_medium=email#!forum/ccp-petmr-codebot/topics - [CCPPETMR/SIRF] matlab libload_*.m better errors (#304) [2 Updates] http://groups.google.com/group/ccp-petmr-codebot/t/1d580562f170ee77 - [CCPPETMR/SIRF] NiftiImageData3DTensor construtor from array not accessible (#303) [5 Updates] http://groups.google.com/group/ccp-petmr-codebot/t/cbba959c809ed3d7 - [CCPPETMR/SIRF] matlab: error catching in libload_*.m (#302) [3 Updates] http://groups.google.com/group/ccp-petmr-codebot/t/442009d55f083485 - [CCPPETMR/SIRF-SuperBuild] Gadgetron tests fail but Travis stays green (#192) [4 Updates] http://groups.google.com/group/ccp-petmr-codebot/t/ceff2cb9c533b386 - [CCPPETMR/SIRF] Travis failing for linux (#300) [1 Update] http://groups.google.com/group/ccp-petmr-codebot/t/c5109ed8e472ad3f - [CCPPETMR/SIRF-SuperBuild] added missing CMake FFTW variables to External_SIRF.cmake, fixes #197 (together with installing boost_1_65_1) (5b8cb89) [1 Update] http://groups.google.com/group/ccp-petmr-codebot/t/c85e86567f89e791 - [CCPPETMR/SIRF] Travis use conda itk on OSX (#295) [3 Updates] http://groups.google.com/group/ccp-petmr-codebot/t/65e721956e918868 - [CCPPETMR/SIRF] Add getter for number of total voxels to NiftiImageData (#301) [6 Updates] http://groups.google.com/group/ccp-petmr-codebot/t/b7e60cc0d7761f55 - [CCPPETMR/SIRF] NiftiImageData template arguments other than float don't compile (#299) [1 Update] http://groups.google.com/group/ccp-petmr-codebot/t/41dfa80d14f8c8a0 ============================================================================ Topic: [CCPPETMR/SIRF] matlab libload_*.m better errors (#304) URL: http://groups.google.com/group/ccp-petmr-codebot/t/1d580562f170ee77 ============================================================================ ---------- 1 of 2 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 04:26PM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172a374c283942 You can view, comment on, or merge this pull request online at: https://github.com/CCPPETMR/SIRF/pull/304 -- Commit Summary -- * matlab libload_*.m better errors -- File Changes -- ---------- 2 of 2 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 06:02PM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/183c4d3bc29341 Merged #304 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ============================================================================ Topic: [CCPPETMR/SIRF] NiftiImageData3DTensor construtor from array not accessible (#303) URL: http://groups.google.com/group/ccp-petmr-codebot/t/cbba959c809ed3d7 ============================================================================ ---------- 1 of 5 ---------- From: Johannes Mayer <[log in to unmask]> Date: Feb 05 08:10AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172952ff3123a4 @rijobro The constructor ``` template<class inputType> NiftiImageData3DDisplacement(const inputType * const data, const VoxelisedGeometricalInfo3D &geom) ---------- 2 of 5 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 08:32AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172a8bbec30403 Yes, I though about this the other day. When creating an image from geometrical info, we actually call this method: `create_from_geom_info`. We could add an argument (`bool is_tensor=false`). ---------- 3 of 5 ---------- From: Johannes Mayer <[log in to unmask]> Date: Feb 05 08:33AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172a99bb2a9891 I'm very happy with it! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ---------- 4 of 5 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 05:24PM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172d60318406ad Ok, pushed to master. I also added `ctests` for `NiftiImageData3DTensor`, `NiftiImageData3DDisplacement` and `NiftiImageData3DDeformation` (as I'd already done for `NiftiImageData` and ---------- 5 of 5 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 10:02AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/183c44535326e6 Ignore bottom comment, managed to do resampling of tensor images too. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ============================================================================ Topic: [CCPPETMR/SIRF] matlab: error catching in libload_*.m (#302) URL: http://groups.google.com/group/ccp-petmr-codebot/t/442009d55f083485 ============================================================================ ---------- 1 of 3 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 06:40AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172474df05c30f loading libraries in matlab currently looks like this: ``` try [notfound, warnings] = loadlibrary('miutilities'); catch error('mutilities library failed to load\n') ---------- 2 of 3 ---------- From: Kris Thielemans <[log in to unmask]> Date: Feb 05 08:19AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1729d74a45319c yes. let's remove `try/catch`. It will error out just like the current code, but at least we'll see why. -- You are receiving this because you are subscribed to this thread. Reply to this email ---------- 3 of 3 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 08:26AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172a3babe2d6c1 ok, see PR #304. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ============================================================================ Topic: [CCPPETMR/SIRF-SuperBuild] Gadgetron tests fail but Travis stays green (#192) URL: http://groups.google.com/group/ccp-petmr-codebot/t/ceff2cb9c533b386 ============================================================================ ---------- 1 of 4 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 06:54AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/17252cae24805e Reopened #192. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ---------- 2 of 4 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 06:55AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/17254533303bde This has started happening again. Most recent error that should be causing it to be red happened during the `cmake` of SIRF: ``` CMake Error at ---------- 3 of 4 ---------- From: Edoardo Pasca <[log in to unmask]> Date: Feb 05 03:51PM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/172854664f6497 on what OS? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ---------- 4 of 4 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 04:19PM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1729d92f7c6d7f both unfortunately... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ============================================================================ Topic: [CCPPETMR/SIRF] Travis failing for linux (#300) URL: http://groups.google.com/group/ccp-petmr-codebot/t/c5109ed8e472ad3f ============================================================================ ---------- 1 of 1 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 06:52AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1725197cc193c2 Due to https://github.com/CCPPETMR/SIRF-SuperBuild/commit/5b8cb8961a8db4376c67528abc98fc6442aad6b2 -- You are receiving this because you are subscribed to this thread. Reply to this email ============================================================================ Topic: [CCPPETMR/SIRF-SuperBuild] added missing CMake FFTW variables to External_SIRF.cmake, fixes #197 (together with installing boost_1_65_1) (5b8cb89) URL: http://groups.google.com/group/ccp-petmr-codebot/t/c85e86567f89e791 ============================================================================ ---------- 1 of 1 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 06:51AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/17250d8f0f791e I think this commit has broken SIRF's Travis (https://travis-ci.org/CCPPETMR/SIRF). The superbuild is also failing but returning green (a probem in itself). I'm not sure what the problem is with ============================================================================ Topic: [CCPPETMR/SIRF] Travis use conda itk on OSX (#295) URL: http://groups.google.com/group/ccp-petmr-codebot/t/65e721956e918868 ============================================================================ ---------- 1 of 3 ---------- From: Kris Thielemans <[log in to unmask]> Date: Feb 05 02:17AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/17161560889484 @rijobro, try doing ```sh DYLD_PRINT_LIBRARIES=YES ctest -VV ``` [see here](https://apple.stackexchange.com/questions/247227/how-to-find-which-shared-library-is-loaded-by-a-process-on-osx) -- ---------- 2 of 3 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 10:23AM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1716628c3bef37 @rijobro pushed 1 commit. 6b051fb547f9fe4858a98f67ec5369558cf19537 print library dependencies for debugging -- You are receiving this because you are subscribed to this thread. ---------- 3 of 3 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 02:43PM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/17249e0b0ac241 Gave this a go, but currently blocked by #300. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ============================================================================ Topic: [CCPPETMR/SIRF] Add getter for number of total voxels to NiftiImageData (#301) URL: http://groups.google.com/group/ccp-petmr-codebot/t/b7e60cc0d7761f55 ============================================================================ ---------- 1 of 6 ---------- From: Johannes Mayer <[log in to unmask]> Date: Feb 05 12:33AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/171067c64151c9 This function is useful to loop over the elements. You can view, comment on, or merge this pull request online at: https://github.com/CCPPETMR/SIRF/pull/301 -- Commit Summary -- ---------- 2 of 6 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 02:09AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1715a9b3905c8f Merged #301 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ---------- 3 of 6 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 10:12AM URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1715c9d8831fb5 Merged. Just for interest, I was using `get_raw_nifti_sptr()->nvox` before. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on ---------- 4 of 6 ---------- From: Kris Thielemans <[log in to unmask]> Date: Feb 05 02:12AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1715ceb484ca9f can I ask how you're using this? If we need it, we should have a virtual function higher up (which could just get it from the geo_info) -- You are receiving this because you are subscribed to ---------- 5 of 6 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 02:20AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/171642ae94f44b Apologies for merging, do you want me to revert? Although we should be using geom_info, I don't fully trust it quite yet (e.g., varying slice distances for MR images). Should we make this an issue ---------- 6 of 6 ---------- From: Johannes Mayer <[log in to unmask]> Date: Feb 05 02:22AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1716533e5a5189 I decided to load the segmentation input of my simulation into a 'NiftiImageData3D' (no modality attached, just geometry and image information). At some points I want to loop over all the pixels in ============================================================================ Topic: [CCPPETMR/SIRF] NiftiImageData template arguments other than float don't compile (#299) URL: http://groups.google.com/group/ccp-petmr-codebot/t/41dfa80d14f8c8a0 ============================================================================ ---------- 1 of 1 ---------- From: Richard Brown <[log in to unmask]> Date: Feb 05 02:08AM -0800 URL: http://groups.google.com/group/ccp-petmr-codebot/msg/171590ed3472be Can I close this? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: -- You have received this digest because you're subscribed to updates for this group. You can change your settings on the group membership page: https://groups.google.com/forum/?utm_source=digest&utm_medium=email#!forum/ccp-petmr-codebot/join . To unsubscribe from this group and stop receiving emails from it, send an email to [log in to unmask] ######################################################################## To unsubscribe from the CCP-PETMR-DEVEL list, click the following link: https://www.jiscmail.ac.uk/cgi-bin/webadmin?SUBED1=CCP-PETMR-DEVEL&A=1