[log in to unmask] Google Groups
Today's topic summary
View all topics
[CCPPETMR/SIRF] [CMake] Use CMake's FindMatlab.cmake as opposed to our own (#47)
Kris Thielemans <[log in to unmask]>: Jul 16 05:42PM -0700

The version supplied since CMake 3.3 seems to work fine, so we set this now as minimum CMake version. This should be fine as the SuperBuild needs 3.7 anyway.
 
Note that it uses Matlab_ROOT_DIR, not ...more
Kris Thielemans <[log in to unmask]>: Jul 16 05:44PM -0700

Needs testing on multiple platforms. don't merge yet! Add a comment where tested.
 
I've tested on CentOS7 (UCL CS system)
 
--
You are receiving this because you are subscribed to this thread. ...more
Back to top
[CCPPETMR/SIRF] for PET demos, run_all executes my_osmaposl (#46)
Kris Thielemans <[log in to unmask]>: Jul 16 05:20PM -0700

As `run_all` just executes all .m files, it also runs `my_osmaposl`, which is actually a function used by one of the demos. This results in a confusing `failed` message.
 
--
You are receiving this ...more
Back to top
[CCPPETMR/SIRF] examples/Matlab/PET/my_osmaposl.m needs a documentation string etc (#45)
Kris Thielemans <[log in to unmask]>: Jul 17 12:19AM

it needs a small preamble (and license info)
 
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: ...more
Back to top
[CCPPETMR/SIRF] pet_data_path.m et al. need to use a global variable (#44)
Kris Thielemans <[log in to unmask]>: Jul 17 12:16AM

`pet_data_path` checks for `SIRF_PET_DATA_PATH` but this will never work. If we want this, we would need
to add
```
global SIRF_PET_DATA_PATH
```
Its documentation will need to mention this ...more
Back to top
[CCPPETMR/SIRF-SuperBuild] previous install can cause compilation problems for newer versions (#28)
Kris Thielemans <[log in to unmask]>: Jul 16 03:56PM -0700

external packages are installed (by default in `INSTALL`, and therefore include files in `INSTALL/include`). This works fine at first build. However, when the include files of one of the external ...more
Kris Thielemans <[log in to unmask]>: Jul 16 11:01PM

STIR bug filed in https://github.com/UCL/STIR/issues/85
 
 
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: ...more
Back to top
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].