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