Potential tensor extraction issue

gbarisano / alps

Bash script to compute DTI-ALPS

31 stars 6 forks source link

Potential tensor extraction issue #4

Closed qtalwaysrun closed 7 months ago

qtalwaysrun commented 7 months ago

Hi,

In the recent commit three weeks ago, I noticed that extraction of tensor components dxx, dyy, dzz was changed from 0, 3, 5 (found at the beginning before the change) to 0, 2, 5 (found at the end after the change). After a test run I found that the final alps values were larger than expected. Since FSL's tensor order is [Dxx, Dxy, Dxz, Dyy, Dyz, Dzz], I was wondering if this was a typo?

Thanks!

gbarisano commented 7 months ago

Hi,

Thank you so much for noticing this. Yes, this was a typo. I apologize. I just corrected it and updated the code.

Thanks again, Giuseppe