andersonwinkler / PALM

PALM: Permutation Analysis of Linear Models
70 stars 28 forks source link

Error in Non-Parametric Combination #60

Closed xuyangwen closed 1 week ago

xuyangwen commented 1 month ago

Dear Anderson,

I am doing a joint inference across four contrasts using the Non-Parametric Combination method in PALM. The following is the configuration file for PALM.

# Configuration file for PALM.
# Version alpha119, running in MATLAB 24.1.0.2537033 (R2024a).
# 25-Sep-2024 10:40:25

-i Output/con4d.nii
-m ../../../Mask/GrayMatter/GrayMatterMask.nii
-d Output/DesignMatrix.csv
-t Output/TContrast.csv
-o Output/contrast
-C 3.09
-T
-fdr
-logp
-ise
-accel tail
-n 5000
-savemask
-zstat
-npccon
-twotail

Everything is fine, and it succeeded in producing all the statistical images for all four contrasts. However, the following error is shown when producing the NPC statistics images.

Screenshot 2024-09-25 at 10 31 34

Do you know how to solve this? Thanks a lot!

Best, Yangwen

xuyangwen commented 2 weeks ago

I also raised this issue in the FSL email list, replied by Thomas Nichols: https://www.jiscmail.ac.uk/cgi-bin/wa-jisc.exe?A2=ind2410&L=FSL&O=D&P=69848. It seems it is related to tail approximation. Once I remove "-accel tail", the error disappear.

andersonwinkler commented 2 weeks ago

Hi Yangwen,

Could you share your data? My email is like my username here on GitHub, at gmail. I'll have a look then.

Thanks!

Anderson

andersonwinkler commented 2 weeks ago

Hi Yangwen, Thanks for sending the data. There was indeed a bug. I've just made a commit to Github in which it's hopefully fixed. If the problem persists, please let me know. Thanks! Anderson

xuyangwen commented 1 week ago

Hi Anderson, I have tried the new version from GitHub. The error, indeed, is gone. Thanks a lot! Yangwen