Back to the main page.

Bug 1282 - alleged bug in ft_topoplotCC

Status CLOSED WONTFIX
Reported 2012-01-25 09:00:00 +0100
Modified 2012-07-07 20:25:33 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P3 normal
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Jan-Mathijs Schoffelen - 2012-01-25 09:00:46 +0100

Reported by Karl Doron, see below: Hello, Great! That makes sense. On my second question for ft_topoplotCC, there is a bug using strmatch on lines 191 and 192. You'll likely encounter an error using horzcat at line 199 if the 'exact' argument is not added due to possible comparison of a vector and a scalar, e.g., begindx will find channel A2* when only A2 is needed. begindx = strmatch(beglabel{i}, lay.label,'exact'); endindx = strmatch(endlabel{i}, lay.label,'exact'); Thanks! karl


Jan-Mathijs Schoffelen - 2012-01-25 10:02:55 +0100

this is confirmed just by reading (no code written).


Jan-Mathijs Schoffelen - 2012-01-25 10:07:05 +0100

@ previous comment: I just read the suggested patch. Looking into ft_topoplotCC, there is no call to strmatch on line 191, but a call to strcmp in line 187. strcmp behaves appropriately. Setting this to wontfix and give feedback to Karl.