wanderine / BROCCOLI

BROCCOLI: Software for Fast fMRI Analysis on Many-Core CPUs and GPUs
GNU General Public License v3.0
115 stars 38 forks source link

TFCE is currently turned off error in RandomiseGroupLevel #38

Closed catcamacho closed 7 years ago

catcamacho commented 7 years ago

Hello, I'm new to using BROCCOLI and have just installed it on my Mac. It set up and installed correctly as far as I can tell, but when I try to use the TFCE option for RandomiseGroupLevel, I get a "TFCE is currently turned off!" error. Am I missing something or is TFCE not implemented?

Here is the command I ran: RandomiseGroupLevel /Users/catcamacho/Box/FFnHK-Oddball/dataprocessing/workflows/L2workflow/_cope_cope1/mergecopes/cope1_merged.nii -design /Users/catcamacho/Box/FFnHK-Oddball/dataprocessing/misc/design.mat -contrasts /Users/catcamacho/Box/FFnHK-Oddball/dataprocessing/misc/tcon.con -inferencemode 3 -verbose

Thanks for you help!

wanderine commented 7 years ago

It is implemented, but I didnt manage to make the implementation much quicker than randomise in FSL, so I decided to turn it off for now.

Den 20 juli 2017 19:35 skrev "Cat Camacho" notifications@github.com:

Hello, I'm new to using BROCCOLI and have just installed it on my Mac. It set up and installed correctly as far as I can tell, but when I try to use the TFCE option for RandomiseGroupLevel, I get a "TFCE is currently turned off!" error. Am I missing something or is TFCE not implemented?

Here is the command I ran: RandomiseGroupLevel /Users/catcamacho/Box/FFnHK- Oddball/dataprocessing/workflows/L2workflow/cope cope1/mergecopes/cope1_merged.nii -design /Users/catcamacho/Box/FFnHK- Oddball/dataprocessing/misc/design.mat -contrasts /Users/catcamacho/Box/FFnHK-Oddball/dataprocessing/misc/tcon.con -inferencemode 3 -verbose

Thanks for you help!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/wanderine/BROCCOLI/issues/38, or mute the thread https://github.com/notifications/unsubscribe-auth/AEGryOAe6ew86Wx9tYvMaI1pLsfk1_WMks5sP4_4gaJpZM4Oeb1e .

catcamacho commented 7 years ago

Oh that makes sense. Is it straightforward for me to turn it on temporarily? Even slightly faster than randomise would be nice even if it isn't as major of an improvement as the other command options.

wanderine commented 7 years ago

It needs more testing first, and right now I'm on vacation.

2017-07-20 21:25 GMT+02:00 Cat Camacho notifications@github.com:

Oh that makes sense. Is it straightforward for me to turn it on temporarily? Even slightly faster than randomise would be nice even if it isn't as major of an improvement as the other command options.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/wanderine/BROCCOLI/issues/38#issuecomment-316805035, or mute the thread https://github.com/notifications/unsubscribe-auth/AEGryAeZyplE2zbFR9KfMwj0ECzr6mTGks5sP6mVgaJpZM4Oeb1e .

-- Anders Eklund, PhD

catcamacho commented 7 years ago

Fair enough. Thanks anyway!