Back to the main page.
Bug 181 - unknown method in ft_electroderealign
| Status | CLOSED FIXED |
| Reported | 2010-10-13 08:58:00 +0200 |
| Modified | 2011-01-05 12:01:12 +0100 |
| Product: | FieldTrip |
| Component: | core |
| Version: | unspecified |
| Hardware: | PC |
| Operating System: | Mac OS |
| Importance: | P1 normal |
| Assigned to: | Robert Oostenveld |
| URL: | |
| Tags: | |
| Depends on: | |
| Blocks: | |
| See also: |
Robert Oostenveld - 2010-10-13 08:58:27 +0200
Created attachment 9 email attachement from Arno It is quite easy to reproduce with the cfg attached to this email and it use to work. The problem is with aligning fiducials. Simply load the mat file and type elec3 = ft_electroderealign(cfg); The error is ??? Error using ==> ft_electroderealign at 479 unknown method
Robert Oostenveld - 2010-10-13 08:58:56 +0200
please note that the email attachement is now attached to the bug report
Robert Oostenveld - 2010-10-13 17:36:52 +0200
With the current fieldtrip version I cannot reproduce it, but get another error message >> ft_electroderealign(cfg) converting units from 'mm' to 'cm' ??? Subscripted assignment between dissimilar structures. Error in ==> ft_electroderealign at 166 template(i) = ft_convert_units(template(i), elec.unit); % ensure that the units are consistent with the electrodes With the fieldtrip version included in the latest EEGLAB I can reproduce it >> cd ~/matlab/eeglab9_0_2_3b/ >> cd external/fieldtrip-partial/ >> ft_electroderealign(cfg) using fiducials {'Nz', 'LPA', 'RPA'} mean distance between fiducials prior to realignment 78.414915, after realignment 67.534062 ??? Error using ==> ft_electroderealign at 479 unknown method
Robert Oostenveld - 2010-10-13 17:56:59 +0200
as a side note: in revision r1093 the cfg.method was cleaned up and a cfg.warp was added (for method=template), cfg.method=realignfiducial is now fiducial, backward compatibility support is in place in the current version. The error in the current version is matlab version specific and has to do with adding a field to a structure. I have fixed that using a simple workaround (and will commit soon) There was an error in the handling of the backward compatibility of the cfg-options with checkconfig: renamed was used instead of renamedval. Fixed I committed the changes, the latest revision (r1926) now seems to work correctly on the input provided by Arno.
Robert Oostenveld - 2011-01-05 11:57:07 +0100
selected a long list of resolved bugs from roboos and changed the status into "RESOLVED"