brainstorm3 icon indicating copy to clipboard operation
brainstorm3 copied to clipboard

Leff ratio in dipole scanning and whitened recordings

Open ftadel opened this issue 6 years ago • 6 comments

@jcmosher suggested we change the scaling factor applied in two functions:

1. Dipole scanning (not implemented)

https://github.com/brainstorm-tools/brainstorm3/blob/master/toolbox/process/functions/process_dipole_scanning.m#L124 Factor = sqrt(DataMatP.Leff / sResultP.Leff); Replaced with: Factor = sqrt(sResultP.Leff);

Problem: When testing with the introduction tutorials, the goodness of fit of the dipoles drops to very small values, example for standard condition at 90ms (Leff=193): image

Current code (top) vs. John's suggestions (bottom): image
image

2. Whitened recordings (already implemented)

https://github.com/brainstorm-tools/brainstorm3/blob/master/toolbox/tree/tree_callbacks.m#L3547 Factor = sqrt(DataMat.Leff / ResultsMat.Leff); Replaced with: Factor = sqrt(sResultP.Leff);

This looks more reasonable (but do we want to see Z=20 here?). Below, figures obtained by right-clicking on the MN source link for the standard condition (Leff=193) > Model evaluation > Save whitened data (top=before, bottom=after modification): image
image

@jcmosher @dimitriospantazis please validate whether this is what you want to obtain.

ftadel avatar Jul 25 '19 13:07 ftadel

https://neuroimage.usc.edu/brainstorm/Tutorials/TutDipScan

jcmosher avatar Aug 07 '19 15:08 jcmosher

@jcmosher ??

ftadel avatar Aug 12 '19 13:08 ftadel

@jcmosher @dimitriospantazis @sbaillet If you have time now, can you please have a look at this?

ftadel avatar Apr 27 '20 09:04 ftadel

@jcmosher @dimitriospantazis @sbaillet Do you just want to ignore this question until someone spots a possible bug?

ftadel avatar Nov 09 '20 08:11 ftadel

This issue is still pending....

ftadel avatar Sep 19 '21 07:09 ftadel

@jcmosher @dimitriospantazis @sbaillet

ftadel avatar Jun 08 '22 11:06 ftadel