Back to the main page.

Bug 1165 - ft_electroderealign internally still uses old sens structure

Status VERIFIED FIXED
Reported 2011-11-17 14:57:00 +0100
Modified 2011-11-23 12:06:17 +0100
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 - 2011-11-17 14:57:28 +0100

Solution: Change all accesses of (sens/elec/norm).pnt accordingly. see mail below On 11/17/2011 1:39 PM, Juan Pablo Neira wrote: > Hello > > I am having problems using the 'Interactive method' of the > ft_electroderealign function. After I align the electrodes, I closed > the figure to get the new position, but I got this error. > --------------------------------------------------------------------------- > ??? Reference to non-existent field 'pnt'. > > Error in ==> ft_electroderealign at 489 > norm.chanpos = warp_apply(norm.m, orig.pnt); > --------------------------------------------------------------------------- > I tracked the error and I found that in the general setup of the > function ft_electroderealign, the line: > > ft_preamble trackconfig > > Is changing the name of the cfg.elec.pnt variable for: > > cfg.elec.elecpos > > I think that is the origin or the error, because I am losing the pnt > field. Any suggestions? > > Regards, > > Juan Pablo Neira Vesga > Christian Albrecht Universität zu Kiel


Jörn M. Horschig - 2011-11-18 13:39:50 +0100

was apparently already fixed, at least it shows up okay in my repository without having changed it now


Boris Reuderink - 2011-11-23 11:40:07 +0100

Send email to Juan asking to confirm that the problem is indeed resolved.


Jörn M. Horschig - 2011-11-23 12:06:17 +0100

I wonder what drove me to the insance statement that it had already been fixed, but I fixed it together with bug 1132 http://code.google.com/p/fieldtrip/source/diff?spec=svn4792&r=4792&format=side&path=/trunk/ft_electroderealign.m