[Mrtrix-discussion] threshold in csdeconv

[Donald Tournier]

Hi Ben,

Thanks for the bug report. You're absolutely right, that line
shouldn't be there. I've fixed that for future releases. In the
meantime, this shouldn't cause too much trouble for current users -
that threshold really only becomes important when super-resolution
kicks in.

Well spotted!
Thanks,

Donald.


On Tue, May 19, 2009 at 1:49 AM, Jeurissen Ben <Ben.Jeurissen at ua.ac.be> wrote:
> Hi Donald,
>
>
>
> I’m under the impression that there is a slight error in the code of the
> csdeconv command:
>
>
>
> The relative threshold below which the corresponding fibre orientation
> density is assumed to be zero, is set to 0.1 % of the mean FOD amplitude
> instead of 10 % of the mean FOD amplitude (as is suggested in your paper).
>
>
>
> This happens at line 627 of csdeconv.cpp:  sdeconv_common.threshold =
> threshold/100.0;
>
>
>
> Cheers,
>
> Ben
>
> _______________________________________________
> Mrtrix-discussion mailing list
> Mrtrix-discussion at www.nitrc.org
> http://www.nitrc.org/mailman/listinfo/mrtrix-discussion
>
>



-- 
Jacques-Donald Tournier (PhD)
Brain Research Institute, Melbourne, Australia
Tel: +61 (0)3 9496 4078