antsMotionCorr time-shift error

[antsx]

antsMotionCorr time-shift error

[stnava]

hmm ---- see a previous fix/commit below

... relevant lines of code are between

https://github.com/stnava/ANTs/blob/master/Examples/antsMotionCorr.cxx#L1290
https://github.com/stnava/ANTs/blob/master/Examples/antsMotionCorr.cxx#L1306

unless you perhaps are using if ( maptoneighbor ) ?

which is likely not tested.

commit 3f249d9

Author: Brian Avants [email protected]

Date: Fri Sep 14 13:55:40 2012 +0000

BUG: offset of slice by 1

brian

On Thu, Mar 13, 2014 at 12:59 PM, Jeffrey Duda [email protected]:

There appears to be a problem in antsMotionCorr that causes the data from
the first time-point to disappear while data from the last time point
appears twice. So for an image of time-points [1,2,3,4,5,6] the output is
[2,3,4,5,6,6]. The last two output time-points images aren't identical
suggesting that different transforms are being used but applied to the same
input image.

Reply to this email directly or view it on GitHubhttps://github.com//issues/76
.

[jeffduda]

Wrong button. Didn't mean to close that. Anyway, that's the section of code I've been looking at. I'm not (intentionally) using maptoneighbor, though from the documentation it's not clear how to turn that on/off

[stnava]

the default looks like maptoneighbor is true - i just changed that.

previously, you turn it off via -u 1

 -u, --useFixedReferenceImage

      use a fixed reference image instead of the neighor in the time

series.

brian

On Thu, Mar 13, 2014 at 1:32 PM, Jeffrey Duda [email protected]:

Reopened #76 #76.

Reply to this email directly or view it on GitHubhttps://github.com//issues/76
.

[stnava]

also - just tested in bold data and seemed to be ok using the ants
antsMotionCorrExample script

let me know if you have further issues or if my recent commit accidentally
fixed the problem.

brian

On Thu, Mar 13, 2014 at 1:38 PM, brian avants [email protected] wrote:

the default looks like maptoneighbor is true - i just changed that.

previously, you turn it off via -u 1

 -u, --useFixedReferenceImage

      use a fixed reference image instead of the neighor in the time

series.

brian

On Thu, Mar 13, 2014 at 1:32 PM, Jeffrey Duda [email protected]:

Reopened #76 #76.

Reply to this email directly or view it on GitHubhttps://github.com//issues/76
.

[jeffduda]

That seems to have resolved it