Back to the main page.

Bug 2485 - series of small topoplots of cohspctrm does not work

Status CLOSED FIXED
Reported 2014-02-26 16:06:00 +0100
Modified 2014-06-18 12:32:26 +0200
Product: FieldTrip
Component: plotting
Version: unspecified
Hardware: All
Operating System: All
Importance: P5 normal
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Tilmann Sander-Thommes - 2014-02-26 16:06:03 +0100

Lines 334 and 335 in topoplot_common.m read % Create time-series of small topoplots: if ~ischar(cfg.xlim) && length(cfg.xlim)>2 && any(ismember(dimtok, 'time')) The "&& any(ismember(dimtok, 'time'))" prevents the plotting of multiple topoplots if dimtok = 'freq'. A calling sequence for multiple topoplots of "cohspctrm" could be cfg=[]; cfg.xlim= [0:4:70]; % ... cfg.parameter='cohspctrm'; cfg.zlim=[-0.4 0.4]; ft_topoplotTFR(cfg, icoh); ... Is the suppression of multiple topoplots of dimtok = 'freq' really intended ? Best, Till


Jan-Mathijs Schoffelen - 2014-03-06 15:43:21 +0100

Hi Till, I am not sure. Could you upload a snippet of code + data to reproduce this? Best, JM


Tilmann Sander-Thommes - 2014-03-07 13:37:11 +0100

Created attachment 596 test script


Tilmann Sander-Thommes - 2014-03-07 13:37:59 +0100

Created attachment 597 output with one map


Tilmann Sander-Thommes - 2014-03-07 13:43:46 +0100

Hi Jan-Mathijs, I attached a test script, the data for that and a correct multi topoplot of coherences I will e-mail directly to you due to their size. The data itself has no meaning, it is actually a marker file. One map topoplot output I get with: Lines 334 and 335 in topoplot_common.m read % Create time-series of small topoplots: if ~ischar(cfg.xlim) && length(cfg.xlim)>2 && any(ismember(dimtok, 'time')) and the sequence of topoplots with: Lines 334 and 335 in topoplot_common.m read % Create time-series of small topoplots: if ~ischar(cfg.xlim) && length(cfg.xlim)>2 Best, Till


Jan-Mathijs Schoffelen - 2014-03-17 15:46:21 +0100

I uploaded the relevant data onto our filesystem. Hope to be able to have a look soon. Stay tuned...


Tilmann Sander-Thommes - 2014-03-17 16:15:17 +0100

Hi Jan-Mathijs, To save your time, really have a look at the code first: Lines 334 and 335 in topoplot_common.m read % Create time-series of small topoplots: if ~ischar(cfg.xlim) && length(cfg.xlim)>2 && any(ismember(dimtok, 'time')) Why should series of small topoplots be restricted to the case of a time dimension only? Best, Till


Jan-Mathijs Schoffelen - 2014-03-18 20:03:55 +0100

Fair point, Till. I'll change the conditional statement in topoplot_common


Jan-Mathijs Schoffelen - 2014-03-18 20:07:05 +0100

bash-4.1$ svn add test/test_bug2485.m A test/test_bug2485.m bash-4.1$ svn commit -m "enhancement - also allow multiple topos when xparam is freq" private/topoplot_common.m test/test_bug2485.m Sending private/topoplot_common.m Adding test/test_bug2485.m Transmitting file data .. Committed revision 9297.