Print

Print


 ============================================================================
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


  - [SyneRBI/SIRF] added pass through of threshold for recon (#643) [4 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/e989506f3783356
  - [SyneRBI/SIRF] added pass through of threshold for recon (#544) [2 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/2461bfa3e9d320b1
  - [SyneRBI/SIRF] Gpu test (#642) [5 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/a063c86f8cb8e506
  - [SyneRBI/SIRF-SuperBuild] ccache within CMake (#58) [2 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/b84bde745586e805
  - [SyneRBI/SIRF-SuperBuild] update STIR to rel_4.0.0 (#377) [3 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/128cfbaf4d999ab0
  - [SyneRBI/SIRF] Proper in-place algebra (#635) [5 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/8618bd3a11bcc83d
  - [SyneRBI/SIRF] stir.make_Poisson_loglikelihood seems to ignore the model parameter (#567) [1 Update]
    http://groups.google.com/group/ccp-petmr-codebot/t/15dbc889fe2de8e4
  - [SyneRBI/SIRF] occasional Bad file descriptor in cGadgetron (#641) [2 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/d726d5bd72080897
  - [SyneRBI/SIRF-SuperBuild] Current v2.1.0 build fails due to ITK (#380) [3 Updates]
    http://groups.google.com/group/ccp-petmr-codebot/t/fb8e501bfc191f3e


 ============================================================================
Topic: [SyneRBI/SIRF] added pass through of threshold for recon (#643)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/e989506f3783356
 ============================================================================

---------- 1 of 4 ----------
From: Alexander Whitehead <[log in to unmask]>
Date: Apr 27 12:17PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1b3189d163406

a pass through for both the maximum and minimum relative change during osmaposl reconstruction has been added

It was easier and cleaner to abandon the old PR and re add the changes to a new branch

---------- 2 of 4 ----------
From: Alexander Whitehead <[log in to unmask]>
Date: Apr 27 12:24PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1b37f8a180098

Sorry, I should have worked from SyneRBI/master not ALEXJAZZ008008/master

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:

---------- 3 of 4 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 27 02:18PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1b9b6459fa47c

@KrisThielemans approved this pull request.





-- 
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: Kris Thielemans <[log in to unmask]>
Date: Apr 27 02:20PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1b9cbe8f462bd

looks good. let's wait for Travis before merging. squash merge I guess

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:


 ============================================================================
Topic: [SyneRBI/SIRF] added pass through of threshold for recon (#544)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/2461bfa3e9d320b1
 ============================================================================

---------- 1 of 2 ----------
From: Alexander Whitehead <[log in to unmask]>
Date: Apr 27 10:16AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1ac8455b719b4

@ALEXJAZZ008008 pushed 1 commit.

59bb7494b5ad0f76495f95100976ec04a332d21a  Merge branch 'master' of https://github.com/SyneRBI/SIRF into passthrough_threshold_for_recon


-- 

---------- 2 of 2 ----------
From: Alexander Whitehead <[log in to unmask]>
Date: Apr 27 12:15PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1b2ffd2cb3793

Closed #544.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/SyneRBI/SIRF/pull/544#event-3277228019



 ============================================================================
Topic: [SyneRBI/SIRF] Gpu test (#642)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/a063c86f8cb8e506
 ============================================================================

---------- 1 of 5 ----------
From: Richard Brown <[log in to unmask]>
Date: Apr 27 09:46AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1aae43ee185c5

You can view, comment on, or merge this pull request online at:

  https://github.com/SyneRBI/SIRF/pull/642

-- Commit Summary --

  * STIR NP test
  * update test
  * update test

---------- 2 of 5 ----------
From: Richard Brown <[log in to unmask]>
Date: Apr 27 09:48AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1aafb17eb26e0

@rijobro pushed 1 commit.

eb5fcdb8cb76f1d1daaa595ec5e6eec321507c41  niftypet test


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:

---------- 3 of 5 ----------
From: Richard Brown <[log in to unmask]>
Date: Apr 27 09:49AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1ab0a0f74717b

Merged #642 into master.

-- 
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: Kris Thielemans <[log in to unmask]>
Date: Apr 27 09:56AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1ab6ea155eb13

@rijobro, can you rename this file to `NfityPET_test.sh` and provide a `README.md` with some information on when it's suitable?

-- 
You are receiving this because you are subscribed to this

---------- 5 of 5 ----------
From: Richard Brown <[log in to unmask]>
Date: Apr 27 09:59AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1ab8de6d629f7

sure

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/SyneRBI/SIRF/pull/642#issuecomment-620109765



 ============================================================================
Topic: [SyneRBI/SIRF-SuperBuild] ccache within CMake (#58)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/b84bde745586e805
 ============================================================================

---------- 1 of 2 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 27 08:45AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1a785c068fe37

Closed #58.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:

---------- 2 of 2 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 27 08:45AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1a785d2ce8dd3

Won't fix this as it seems up to the depend projects to take care of `ccache`. Developers will have to use the linking trick if they're interested in `ccache` for a project that doesn't have support


 ============================================================================
Topic: [SyneRBI/SIRF-SuperBuild] update STIR to rel_4.0.0 (#377)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/128cfbaf4d999ab0
 ============================================================================

---------- 1 of 3 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 26 11:49PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/18a488628ea05

@KrisThielemans pushed 1 commit.

4160713267dff61c319803c7ff24c3c69db08322  STIR release_4.0 branch


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:

---------- 2 of 3 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 26 11:58PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/18ac63e6d8c2f

Another STIR 4.0.0 bug https://github.com/UCL/STIR/issues/520 caused seg-faults in earlier Travis run. hopefully ok now

-- 
You are receiving this because you are subscribed to this thread.

---------- 3 of 3 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 27 08:43AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1a76affc311cb

@KrisThielemans pushed 2 commits.

c2f7f36650cd1879448f45a31e545b5305537030  Merge remote-tracking branch 'origin/master' into STIR_v4.0.0
c730c375d5351ba4ff11698efd072dce7359599e  update to latest


 ============================================================================
Topic: [SyneRBI/SIRF] Proper in-place algebra (#635)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/8618bd3a11bcc83d
 ============================================================================

---------- 1 of 5 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 02:48AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1941264677157

@evgueni-ovtchinnikov commented on this pull request.



> -    diff = new_acq_data - acq_data
+    diff = new_acq_data
+    diff -= acq_data

just was checking `-=`

-- 

---------- 2 of 5 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 02:58AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/194958b96aa4f

@evgueni-ovtchinnikov commented on this pull request.



> @@ -378,11 +378,13 @@ complex_float_t a, complex_float_t b)
 	int n = y.number();
 	ISMRMRD::Acquisition ax;
 	ISMRMRD::Acquisition

---------- 3 of 5 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 03:03AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/194e2b462eb99

@evgueni-ovtchinnikov commented on this pull request.



> @@ -222,6 +222,7 @@ namespace sirf {
 		// abstract methods
 
 		virtual void empty() = 0;
+		virtual void

---------- 4 of 5 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 03:12AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1955ff1549a5b

@evgueni-ovtchinnikov pushed 2 commits.

f0a3e757ff2c86f522bb22a66849fbad2764634c  moved in-place algebra tests from examples to tests
06b289d505dbbd19d98b9da85576e5daef3c5bec  Merge branch

---------- 5 of 5 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 07:06AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1a226c35ab126

Merged #635 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:


 ============================================================================
Topic: [SyneRBI/SIRF] stir.make_Poisson_loglikelihood seems to ignore the model parameter (#567)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/15dbc889fe2de8e4
 ============================================================================

---------- 1 of 1 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 05:11AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/19bd8c039c96a

Closed #567 via fd573010d15f373799c2492fdba4e7306272d5ea.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:


 ============================================================================
Topic: [SyneRBI/SIRF] occasional Bad file descriptor in cGadgetron (#641)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/d726d5bd72080897
 ============================================================================

---------- 1 of 2 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 27 04:22AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/199292278593c

This job https://travis-ci.org/github/SyneRBI/SIRF-SuperBuild/jobs/679959452#L16334
from https://github.com/SyneRBI/SIRF-SuperBuild/pull/377 (which is a `DEVEL` build) fails, while others are fine.

---------- 2 of 2 ----------
From: Evgueni Ovtchinnikov <[log in to unmask]>
Date: Apr 27 04:43AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/19a58e49d3f2d

I have seen this error message in Travis logs many times, but the reported error never ever happened locally, so impossible to investigate, I am afraid.

-- 
You are receiving this because you are


 ============================================================================
Topic: [SyneRBI/SIRF-SuperBuild] Current v2.1.0 build fails due to ITK (#380)
URL: http://groups.google.com/group/ccp-petmr-codebot/t/fb8e501bfc191f3e
 ============================================================================

---------- 1 of 3 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 26 11:33PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/1896e8835f52a

why do you say this? It works for me. Their homepage still points to this repo

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on

---------- 2 of 3 ----------
From: Edoardo Pasca <[log in to unmask]>
Date: Apr 26 11:47PM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/18a2af174bb55

I say it because it failed more times for me and I was forced to change the repo to make it work.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email

---------- 3 of 3 ----------
From: Kris Thielemans <[log in to unmask]>
Date: Apr 27 12:00AM -0700
URL: http://groups.google.com/group/ccp-petmr-codebot/msg/18ae090b37f88

weird that you had problems. Close this?

For the record, what did you change it to?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view





--
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 SYNERBI-DEVEL list, click the following link:
https://www.jiscmail.ac.uk/cgi-bin/webadmin?SUBED1=SYNERBI-DEVEL&A=1