search

brainlife / app-life

LiFE (Linear Fasicle Evaluation) predicts the measured diffusion signal using the orientation of the fascicles present in a connectome. LiFE uses the difference between the measured and predicted diffusion signals to measure prediction error. The connectome model prediction error is used to compute two metrics to evaluate the evidence supporting properties of the connectome.
1 stars 5 forks source link

Does LiFE use t1 input? #2

Closed soichih closed 7 years ago

soichih commented 7 years ago

app-life currently requires t1 as one of the input file. @kitchell and I believe that it's not used by LiFE, or that it works without it. We ran LiFE with bogus path to t1 and LiFE finished successfully.

@francopestilli Please let me know if we should get rid of t1 input - this simplifies our workflow and make it easier for user to submit LiFE.

francopestilli commented 7 years ago

@soichih not necessarily. But it is of help for making plots of the anatomy. If the T1 is not passed the XFORM from the T1 would be necessary. Now we extract that information from the T1 during the ENCODE stage.

soichih commented 7 years ago

@francopestilli

I've looked at the latest encode code, and I couldn't find any code that's using fe.path.anatomy once it's set by feSet - as far as I can tell. There might be some matlab magic happening somewhere, but xform seems to be always set from dwiFile / qto_xyz.

Can you point me to the code where T1 is actually read by encode?

francopestilli commented 7 years ago

Let me check when back.

Franco

sent via phone, pardon brevity and mistakes.

On Aug 16, 2017 5:55 PM, "Soichi Hayashi" notifications@github.com wrote:

@francopestilli https://github.com/francopestilli

I've looked at the latest encode code, and I couldn't find any code that's using fe.path.anatomy once it's set by feSet - as far as I can tell. There might be some matlab magic happening somewhere, but xform seems to be always set from dwiFile / qto_xyz.

Can you point me to the code where T1 is actually read by encode?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/brain-life/app-life/issues/2#issuecomment-322817362, or mute the thread https://github.com/notifications/unsubscribe-auth/ACBYc22waJ9QIHxDKulO_ceHO_6LHkI2ks5sYxEGgaJpZM4O1QkU .

soichih commented 7 years ago

I've removed the t1 input from app-life for the time being.