Back to the main page.

Bug 2993 - The ft_electroderealign GUI does not close and save the changes

Status CLOSED WORKSFORME
Reported 2015-10-23 15:23:00 +0200
Modified 2016-06-14 16:18:22 +0200
Product: FieldTrip
Component: forward
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P5 normal
Assigned to: Robert Oostenveld
URL:
Tags:
Depends on:
Blocks:
See also:

Paolo Belardinelli - 2015-10-23 15:23:25 +0200

Created attachment 745 Localite electrode position file OS: Win8 Matlab version: Matlab2015a Fieldtrip version: fieldtrip-20150828, with ft_electroderealign downloaded from https://github.com/fieldtrip/fieldtrip/blob/master/ft_electroderealign.m The following code is employed: cfg = []; cfg.elec = ft_read_sens(‘your localite file’) cfg.method = ‘interactive’; elec_realigned = ft_electroderealign(cfg) cfg = []; cfg.elec = elec_realigned; layout = ft_prepare_layout(cfg); We don't get any proper error. However, after pushing the 'apply' button, we are still not able to save the modified configuration and manually close the interactive window. As a consequence, when visualizing the new layout, we are still plotting the old configuration. Attached, we upload an example of LOCALITE file with electrode positions as acquired before our measurement.


Robert Oostenveld - 2015-11-11 13:16:29 +0100

I cannot reproduce. I did the following elec_original = ft_read_sens('EEGMarkersLocalite.xml'); cfg = []; cfg.elec = elec_original; cfg.method = 'interactive'; elec_realigned1 = ft_electroderealign(cfg); % here I rotated 180 degrees around z assert(~isequal(elec_realigned1.chanpos, elec_original.chanpos)); cfg = []; cfg.elec = elec_original; figure; ft_plot_lay(ft_prepare_layout(cfg)); cfg = []; cfg.elec = elec_original; cfg.rotate = 180; figure; ft_plot_lay(ft_prepare_layout(cfg)); cfg = []; cfg.elec = elec_realigned1; figure; ft_plot_lay(ft_prepare_layout(cfg)); all but the first layout figure (the original) look fine to me.


Robert Oostenveld - 2015-11-11 13:17:40 +0100

Created attachment 753 screen shot of layout figures


Robert Oostenveld - 2015-11-11 13:18:58 +0100

note that I tested with R2015b not with R2015a (which I don't have). Please try again, and perhaps you might want to check whether it is 2015a specific.


Robert Oostenveld - 2015-11-11 13:29:18 +0100

I made some small adjustments along the way and added a test script. mac011> svn commit Sending ft_interactiverealign.m Sending ft_sensorrealign.m Sending plotting/ft_plot_axes.m Adding test/inspect_bug2993.m Transmitting file data .... Committed revision 10869.


Robert Oostenveld - 2016-06-14 16:18:22 +0200

Hereby I am closing several bugs that have been resolved for some time (WONTFIX, WORKSFORME, etc.). If you disagree, please reopen and explain.