Back to the main page.

Bug 1715 - Transpose operation missing in ft_channelcombination.m

Status CLOSED FIXED
Reported 2012-09-18 11:00:00 +0200
Modified 2012-10-24 10:43:55 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: All
Importance: P3 normal
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Bart Gips - 2012-09-18 11:00:30 +0200

Line 59-61 of ft_channelcombination produce a warning when the input cfg.channelcmb contains columns of channel pairs, instead of a rows of channel pairs. (i.e. cfg.channelcmb=2xM instead of Mx2) To fix this, it displays the warning: 'Warning: transposing channelcombination matrix' However, after this, it does not actually transpose the channelcombination matrix at all! Something similar to: channelcmb=channelcmb'; should be added after the warning.


Jan-Mathijs Schoffelen - 2012-09-19 15:52:14 +0200

Thanks for the report. I added the transposition. Could you check whether it works fine now? bash-3.2$ svn commit ft_channelcombination.m Sending ft_channelcombination.m Transmitting file data . Committed revision 6476.


Jan-Mathijs Schoffelen - 2012-09-19 15:52:59 +0200

I don't know whether you already have an svn account, but if you are working from the fieldtrip version at /home/common/matlab, it takes ~20 minutes before it's synchronized to the latest version of the repository


Bart Gips - 2012-09-19 16:50:11 +0200

(In reply to comment #2) I have used the read-only google svn account. It works fine now.