Print

Print


[log in to unmask] Google Groups
Today's topic summary
View all topics
[SyneRBI/SIRF] added pass through of threshold for recon (#643)
Alexander Whitehead <[log in to unmask]>: Apr 27 12:17PM -0700

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 ...more
Alexander Whitehead <[log in to unmask]>: Apr 27 12:24PM -0700

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: ...more
Kris Thielemans <[log in to unmask]>: Apr 27 02:18PM -0700

@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: ...more
Kris Thielemans <[log in to unmask]>: Apr 27 02:20PM -0700

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: ...more
Back to top
[SyneRBI/SIRF] added pass through of threshold for recon (#544)
Alexander Whitehead <[log in to unmask]>: Apr 27 10:16AM -0700

@ALEXJAZZ008008 pushed 1 commit.
 
59bb7494b5ad0f76495f95100976ec04a332d21a Merge branch 'master' of https://github.com/SyneRBI/SIRF into passthrough_threshold_for_recon
 
 
--
...more
Alexander Whitehead <[log in to unmask]>: Apr 27 12:15PM -0700

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
...more
Back to top
[SyneRBI/SIRF] Gpu test (#642)
Richard Brown <[log in to unmask]>: Apr 27 09:46AM -0700

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 ...more
Richard Brown <[log in to unmask]>: Apr 27 09:48AM -0700

@rijobro pushed 1 commit.
 
eb5fcdb8cb76f1d1daaa595ec5e6eec321507c41 niftypet test
 
 
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
...more
Richard Brown <[log in to unmask]>: Apr 27 09:49AM -0700

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: ...more
Kris Thielemans <[log in to unmask]>: Apr 27 09:56AM -0700

@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 ...more
Richard Brown <[log in to unmask]>: Apr 27 09:59AM -0700

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
...more
Back to top
[SyneRBI/SIRF-SuperBuild] ccache within CMake (#58)
Kris Thielemans <[log in to unmask]>: Apr 27 08:45AM -0700

Closed #58.
 
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: ...more
Kris Thielemans <[log in to unmask]>: Apr 27 08:45AM -0700

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 ...more
Back to top
[SyneRBI/SIRF-SuperBuild] update STIR to rel_4.0.0 (#377)
Kris Thielemans <[log in to unmask]>: Apr 26 11:49PM -0700

@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: ...more
Kris Thielemans <[log in to unmask]>: Apr 26 11:58PM -0700

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. ...more
Kris Thielemans <[log in to unmask]>: Apr 27 08:43AM -0700

@KrisThielemans pushed 2 commits.
 
c2f7f36650cd1879448f45a31e545b5305537030 Merge remote-tracking branch 'origin/master' into STIR_v4.0.0
c730c375d5351ba4ff11698efd072dce7359599e update to latest ...more
Back to top
[SyneRBI/SIRF] Proper in-place algebra (#635)
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 02:48AM -0700

@evgueni-ovtchinnikov commented on this pull request.
 
 
 
> - diff = new_acq_data - acq_data
+ diff = new_acq_data
+ diff -= acq_data
 
just was checking `-=`
 
--
...more
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 02:58AM -0700

@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 ...more
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 03:03AM -0700

@evgueni-ovtchinnikov commented on this pull request.
 
 
 
> @@ -222,6 +222,7 @@ namespace sirf {
// abstract methods

virtual void empty() = 0;
+ virtual void ...more
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 03:12AM -0700

@evgueni-ovtchinnikov pushed 2 commits.
 
f0a3e757ff2c86f522bb22a66849fbad2764634c moved in-place algebra tests from examples to tests
06b289d505dbbd19d98b9da85576e5daef3c5bec Merge branch ...more
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 07:06AM -0700

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: ...more
Back to top
[SyneRBI/SIRF] stir.make_Poisson_loglikelihood seems to ignore the model parameter (#567)
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 05:11AM -0700

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: ...more
Back to top
[SyneRBI/SIRF] occasional Bad file descriptor in cGadgetron (#641)
Kris Thielemans <[log in to unmask]>: Apr 27 04:22AM -0700

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. ...more
Evgueni Ovtchinnikov <[log in to unmask]>: Apr 27 04:43AM -0700

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 ...more
Back to top
[SyneRBI/SIRF-SuperBuild] Current v2.1.0 build fails due to ITK (#380)
Kris Thielemans <[log in to unmask]>: Apr 26 11:33PM -0700

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 ...more
Edoardo Pasca <[log in to unmask]>: Apr 26 11:47PM -0700

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 ...more
Kris Thielemans <[log in to unmask]>: Apr 27 12:00AM -0700

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 ...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].


To unsubscribe from the SYNERBI-DEVEL list, click the following link:
https://www.jiscmail.ac.uk/cgi-bin/webadmin?SUBED1=SYNERBI-DEVEL&A=1