Back to the main page.

Bug 3005 - ft_rejectvisual: summary mode doesn't work for single trials

Status CLOSED FIXED
Reported 2015-11-13 22:21:00 +0100
Modified 2016-06-14 16:14:55 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P5 normal
Assigned to: Robert Oostenveld
URL:
Tags:
Depends on:
Blocks:
See also: http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=1474http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=3012http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=2782

Teresa Madsen - 2015-11-13 22:21:47 +0100

The problem is nearly identical to Bug 1474 (http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=1474), but about single trials instead of single channels. Running: cfg = []; cfg.method = 'summary'; data = ft_rejectvisual(cfg,data); where data is continuous, i.e., only 1 trial, results in the following error: Error using matlab.graphics.axis.Axes/set While setting the 'YLim' property of Axes: Value must be a 1x2 vector of numeric type in which the second element is larger than the first and may be Inf Error in axis>LocSetLimits (line 200) set(ax,... Error in axis (line 93) LocSetLimits(ax(j),cur_arg); Error in rejectvisual_summary>redraw (line 309) axis([0.5 xmax+0.5 ymin ymax]); Error in rejectvisual_summary (line 133) redraw(h); Error in ft_rejectvisual (line 275) [chansel, trlsel, cfg] = rejectvisual_summary(cfg, tmpdata); I've updated my local copy to use ymin-0.5 and ymax+0.5, and it works fine.


Teresa Madsen - 2015-12-01 17:46:34 +0100

(In reply to Teresa Madsen from comment #0) Side note: The same error occurs if you toggle off all trials but one in initially multi-trial data.


Robert Oostenveld - 2015-12-03 11:49:01 +0100

mac011> svn commit private/rejectvisual_summary.m test/inspect_bug3005.m Sending private/rejectvisual_summary.m Adding test/inspect_bug3005.m Transmitting file data .. Committed revision 10961. this fixes it, please see test script.


Robert Oostenveld - 2016-06-14 16:14:55 +0200

Hereby I am closing multiple bugs that have been resolved for some time now. If you don't agree to the resolution, please reopen.