[log in to unmask]

Google Groups

Kris Thielemans <[log in to unmask]>: Jun 05 03:20PM -0700 also occasional license   closes #660 You can view, comment on, or merge this pull request online at:   https://github.com/SyneRBI/SIRF/pull/707   -- Commit Summary --   * updated copyright ...more

Kris Thielemans <[log in to unmask]>: Jun 05 08:20AM -0700 https://travis-ci.org/github/SyneRBI/SIRF/builds/694620965 has 2 OSX jobs failing like this ``` File "/Users/travis/build/SyneRBI/SIRF/src/xGadgetron/pGadgetron/tests/test2.py", line 70, in ...more

Kris Thielemans <[log in to unmask]>: Jun 05 08:50AM -0700 Closed #705 via #706.   -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ...more

Biguri <[log in to unmask]>: Jun 05 08:24AM -0700 Fix #705 You can view, comment on, or merge this pull request online at:   https://github.com/SyneRBI/SIRF/pull/706   -- Commit Summary --   * Relaxed gadgetron test as refered in #705 ...more

Kris Thielemans <[log in to unmask]>: Jun 05 08:50AM -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]>: Jun 05 08:50AM -0700 Merged #706 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]>: Jun 05 08:38AM -0700 Closed #37 via 3b617123bb87c9134f4467423e69b013c5f6fb54.   -- 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]>: Jun 05 08:30AM -0700 Closed #690 via b19ce4173c75fdb7c088fe6909b00cea5e29d45b.   -- 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]>: Jun 05 08:22AM -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]>: Jun 05 08:22AM -0700 Merged #703 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]>: Jun 05 08:21AM -0700 Merged #702 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]>: Jun 05 08:21AM -0700 Closed #700 via #702.   -- 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]>: Jun 05 08:15AM -0700 ``` VER=2.2.0 ``` 1. Submodules (within SIRF) - [ ] tag data + `git submodule update` # **should not complain** + `cd data && git tag -a v$VER -m "version $VER" && git push --tags && ...more

Kris Thielemans <[log in to unmask]>: Jun 05 08:10AM -0700 Closed #647.   -- 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/issues/647#event-3411793441 ...more

Kris Thielemans <[log in to unmask]>: Jun 05 03:02PM fine. Maybe Nifti doesn't just copy the data, but that loop could be optimised at some point. Or indeed we give up on having "time" in a certain way as well. I still think we shouldn't pretend to the ...more

Johannes Mayer <[log in to unmask]>: Jun 05 07:19AM -0700 Had to change the call to as_array() for the coildata to get the MR notebooks working.     You can view, comment on, or merge this pull request online at: ...more

Kris Thielemans <[log in to unmask]>: Jun 05 07:40AM -0700 Merged #53 into master.   -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ...more

Johannes Mayer <[log in to unmask]>: Jun 05 07:18AM -0700 The undersampled reconstruction notebook needs this package but it&#39;s not installed in the current VM. You can view, comment on, or merge this pull request online at: ...more

Kris Thielemans <[log in to unmask]>: Jun 05 07:38AM -0700 Merged #168 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]>: Jun 05 02:19AM -0700 Here's some lines that can help with doing this, thanks to StackExchange. Below assumes you have `sh` or `bash` (so presumably you are on Linux/MacOS)   First step was to update ...more

Kris Thielemans <[log in to unmask]>: Jun 05 02:20AM -0700 I don't think there's any need in going overboard with add copyright to all files, but it should be there and up-to-date for most of the source files.   Can we all remember to update the copyright ...more

Casper da Costa-Luis <[log in to unmask]>: Jun 05 02:29AM -0700 hah, sure - feel free to add an example below https://github.com/casperdcl/git-fame#codeowners but   - no sane person wants to read a massive copyright notice at the top of every single source file ...more

Kris Thielemans <[log in to unmask]>: Jun 05 02:44AM -0700 > a maximum of 5 lines (upto 80 chars each) of metadata at the top of each file is acceptable.   I have no idea really. I suggest to follow https://www.apache.org/licenses/LICENSE-2.0.html#apply   ...more

Richard Brown <[log in to unmask]>: Jun 05 01:53AM -0700 Merged #407 into master.   -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ...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].