Back to the main page.

Bug 1143 - begsample and endsample not processed for gdf in ft_read_event

Status CLOSED FIXED
Reported 2011-11-11 12:28:00 +0100
Modified 2012-06-20 15:03:22 +0200
Product: FieldTrip
Component: fileio
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P3 normal
Assigned to: Boris Reuderink
URL:
Tags:
Depends on:
Blocks:
See also: http://bugzilla.fcdonders.nl/show_bug.cgi?id=1281

Andrei Belitski - 2011-11-11 12:28:48 +0100

Hi guys, small thing: it seems that I can't specify a sample range to process events of a gdf file. In ft_read_event.m on line 364 begsample and endsample are not passed to the read_trigger function that created an out of memory error, as my gdf is rather large and the evoked biosig toolbox command tries to read it all at once. so I fixed it in my local copy by providing those parameters. thanks Andrei


Robert Oostenveld - 2011-11-11 15:31:03 +0100

Can you send the change that you made to the function with the approximate line number? As diff, patch, or just copy and paste the section you changed.


Andrei Belitski - 2011-11-11 16:20:19 +0100

Created attachment 186 the modified ft_read_event.m file to handle sample range for gdf format I only modified line 364 to add the 'minsample' and 'maxsample' also note: it seems that most other functions like ft_read_data or read_trigger use 'begsample' and 'endsample' to specify a range, whereas in ft_read_event the parameters are called 'minsample' and 'maxsample', may be would nice to make it consistent to avoid confusion thanks


Robert Oostenveld - 2011-11-11 16:39:15 +0100

> also note: it seems that most other functions like ft_read_data or read_trigger > use 'begsample' and 'endsample' to specify a range, whereas in ft_read_event > the parameters are called 'minsample' and 'maxsample', may be would nice to > make it consistent to avoid confusion That is indeed something to consider. The full specification of event selection (which is especially relevant for inline realtime systems) is defined in fileio/ft_filter_event.


Boris Reuderink - 2012-01-17 12:21:09 +0100

Added time estimate.


Boris Reuderink - 2012-02-01 17:30:17 +0100

I have incorporated your fix, see SVN revision 5223 (note that I referred to the wrong bug in the commit msg!). @Andrei, could you verify that the changed code does indeed work as you intended?


Boris Reuderink - 2012-02-01 17:36:05 +0100

(In reply to comment #5) BTW, please note that *tonight* (2012-2-1) there will be no fieldtrip release related to the huge number of fixes that we made. So, please use the version of 2012-2-2 for verification.


Robert Oostenveld - 2012-02-04 10:50:42 +0100

(In reply to comment #6) Following the bug-binge, the last few days we have not had any internal issues with our "beta testers". So I have re-enabled the automatic ftp release script.


Boris Reuderink - 2012-06-20 15:03:22 +0200

Changed my resolved bugs to closed.