View Issue Details

IDProjectCategoryView StatusLast Update
0003559Slicer4Core: Diffusion and Tractography (Any modules)public2014-07-22 19:29
Reporterkikinis Assigned Toreckbo  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.3.1-2 
Target VersionSlicer 4.4.0Fixed in VersionSlicer 4.4.0 
Summary0003559: Glyphs are ellipsiod disks perpendicular to the fiber tracts
Description

From Ron: "I have observed it with two completely different data
sets, so I think it is a bug in the program. I posted one of the DWI
data sets in the Slicer data store. Its called PerthDWITest in the
sample data and tests category."

Screenshot attached.

Additional Information

"I re-ran UKF and basically got working results. For the data set I am
using, 1 seed per voxel runs for about 2 hours and produces an mrb of
roughly 1.5GB. 8 seeds per voxel crashes my machine within about 6
hours. Nighlty build 1-10 on Mac Mavericks with 16GB of Ram and 8 cores.
When I look at the right cortico-spinal tract, I see a lot of
"broken-off" streamlines, where I would expect them to continue based on
my anatomical knowledge" - Ron

TagsNo tags attached.

Activities

2014-01-14 06:45

 

2014-01-14 07:32

 

kikinis

kikinis

2014-01-14 07:32

developer   ~0010515

I just added the rCST image

kikinis

kikinis

2014-01-14 07:36

developer   ~0010516

From: Lauren O'Donnell
In case this helps: it's probably that the tensors are not rotated into
RAS space before saving the file.

From: Johnson, Hans J
The file should be in LPS orientation to match the dicom data that it
came from. If conversion needs to Occur, it needs to happen in Slicer.
This does sound like the issue.
Hans

From Johnson, Hans J :
Ryan: Can you please take a first pass at looking at this? I’m really
swamped this week.
Thanks,
Hans

jcfr

jcfr

2014-05-13 15:30

administrator   ~0011829

From Ryan - Jan 28, 2014:

Hi Hans,

The problem was in vtk_writer.cc, here is the fix:

================
https://github.com/pnlbwh/ukftractography/blob/master/ukf/vtk_writer.cc

{
eigenVec1 <<
state[_tensor_space (tensorNumber - 1) + _p_m1], state[_tensor_space
(tensorNumber - 1) + _p_m2],
state[_tensor_space * (tensorNumber - 1) + _p_m3];

// Perform ijk->RAS transform on eigen vectors
eigenVec1 = _sizeFreeI2R * eigenVec1;

// Renormalize eigenvectors
eigenVec1.normalize();

}

// Compute the diffusion matrix in RAS coordinate system
// The transformed matrix is still positive-definite
mat33_t I;
I <<
1, 0, 0,
0, 1, 0,
0, 0, 1;

D = ((_p_l1 - _p_l2) eigenVec1 eigenVec1.transpose + _p_l2 I) 1e-6;

jcfr

jcfr

2014-05-13 15:30

administrator   ~0011830

From Yogesh - Jan 28. 2014:

Hans,

The problem was that the 2nd and 3rd eigenvectors were not necessarily orthogonal to each other or to the main eigenvector.
The algorithm only estimates the main eigenvector and the other 2 can be easily computed from that one.

This code was in the glyphs section (so, it was not really part of the estimation algorithm, but used only for display).
Hence, Ron was seeing plates instead of ellipsoids in the display.

I hope that the solution we came up with should fix this issue.

Best,
Yogesh

jcfr

jcfr

2014-05-13 15:31

administrator   ~0011831

Fixed by https://github.com/pnlbwh/ukftractography/commit/53b72b9a9623a2f02f85208f2cbba501c0ef3ee7

Issue History

Date Modified Username Field Change
2014-01-14 06:45 reckbo New Issue
2014-01-14 06:45 reckbo File Added: Screen Shot 2014-01-11 at 8.44.46 PM.png
2014-01-14 07:01 jcfr Target Version => Slicer 4.4.0
2014-01-14 07:02 jcfr Reporter reckbo => kikinis
2014-01-14 07:02 jcfr Assigned To => reckbo
2014-01-14 07:02 jcfr Status new => assigned
2014-01-14 07:32 kikinis File Added: Screen Shot 2014-01-14 at 6.19.10 AM.png
2014-01-14 07:32 kikinis Note Added: 0010515
2014-01-14 07:36 kikinis Note Added: 0010516
2014-05-13 15:30 jcfr Note Added: 0011829
2014-05-13 15:30 jcfr Note Added: 0011830
2014-05-13 15:31 jcfr Note Added: 0011831
2014-05-13 15:31 jcfr Status assigned => resolved
2014-05-13 15:31 jcfr Fixed in Version => Slicer 4.4.0
2014-05-13 15:31 jcfr Resolution open => fixed
2014-07-22 19:29 kikinis Status resolved => closed
2016-02-29 13:48 jcfr Category Extension: UKFTractography => Core: Diffusion & Tractography
2016-02-29 14:05 jcfr Category Core: Diffusion & Tractography => Core: Diffusion & Tractography (Any modules)
2016-06-20 14:14 jcfr Category Core: Diffusion & Tractography (Any modules) => Core: Diffusion and Tractography (Any modules)