Back to the main page.
Bug 1599 - fixedori has no influence when using LCMV
Status | CLOSED FIXED |
Reported | 2012-07-05 16:50:00 +0200 |
Modified | 2012-08-23 14:02:12 +0200 |
Product: | FieldTrip |
Component: | core |
Version: | unspecified |
Hardware: | PC |
Operating System: | Windows |
Importance: | P3 normal |
Assigned to: | Jörn M. Horschig |
URL: | |
Tags: | |
Depends on: | |
Blocks: | |
See also: |
Jörn M. Horschig - 2012-07-05 16:50:18 +0200
next one :) fixedori is set correctly in beamformer_lcmv, the only problem is that sourceanalysis seems to not bother about this (or, as JM would probably say: it's a very stubborn beast!) %% draft of example script % load your very favourite data data = load(...); vol = load(...); % set basic cfg options cfg = []; cfg.vol = vol; cfg.method = 'lcmv'; cfg.fixedori ='yes'; oriyes = ft_sourceanalysis(cfg, data); cfg.fixedori = 'no'; orino = ft_sourceanalysis(cfg, data); % cfg is of course different, see above, so remove it orino = rmfield(orino, 'cfg'); oriyes = rmfield(oriyes, 'cfg'); isequal(oriyes, orino); % %ans = 1 %
Jan-Mathijs Schoffelen - 2012-07-09 10:59:27 +0200
Hi Jörn, Please retry with cfg.lcmv.fixedori = 'yes', rather than cfg.fixedori = 'yes'
Jörn M. Horschig - 2012-07-09 13:13:48 +0200
oh right, I forgot to mention, it only happens when using a pre-computed filter See also ft_sourceanalysis line 808 and onwards. tmpdip is computed using beamformer_lcmv (which is perfectly fine), but then dip is constructed without looking for ori (it does not matter whether it's cfg.lcmv.fixedori or cfg.fixedori).
Jörn M. Horschig - 2012-07-25 11:09:26 +0200
aha, I assume ori was not added, because it cannot be estimated for individual trials, however for now, I add the ori field to all single trial estimates. I also made adding a precomputed filter possible when the grid is not predefined (given that dimensions coincide). svn ci ft_sourceanalysis.m test/test_bug1599.m -m "bugfix-#1599-fixedori for lcmv works now also when using a predefined filter" Sending ft_sourceanalysis.m Adding test/test_bug1599.m Transmitting file data ..
Jörn M. Horschig - 2012-07-25 11:39:54 +0200
uh, I just saw this in ft_sourcedescriptives: % The following option only applies to LCMV single-trial timecourses. % cfg.fixedori = 'within_trials' or 'over_trials' (default = 'over_trials') within trials cannot be supported when using a predefined filter, given that all data will be concatenated for more efficient computation and ori will be computed on the concatenated data. something we need to hint to?