Back to the main page.
Bug 2522 - ft_appenddata returns sampleinfo-error when concatenating channels
Status | CLOSED FIXED |
Reported | 2014-04-03 10:52:00 +0200 |
Modified | 2014-05-14 20:08:52 +0200 |
Product: | FieldTrip |
Component: | core |
Version: | unspecified |
Hardware: | PC |
Operating System: | Windows |
Importance: | P5 major |
Assigned to: | Robert Oostenveld |
URL: | |
Tags: | |
Depends on: | |
Blocks: | |
See also: |
Stan van Pelt - 2014-04-03 10:52:21 +0200
When concatenating data over channels using ft_appenddata, an error is returned saying that 'sampleinfo' is missing from the input data. both datasets I put in actually do have the same sampleinfo-fields. When looking in the code, it seems that a line like "data.sampleinfo = varargin{1}.sampleinfo;" is missing around line 254. If I put it in, the code runs fine.
Robert Oostenveld - 2014-04-03 11:08:30 +0200
Hi Stan, It probably relates to a change to the code I made yesterday evening. I will fix it. Robert
Robert Oostenveld - 2014-04-03 11:12:44 +0200
The bug was indeed related to a change in revision 9336 related to bug 2518. I have fixed it. roboos@mentat001> svn commit ft_appenddata.m Sending ft_appenddata.m Transmitting file data . Committed revision 9339.
Robert Oostenveld - 2014-04-03 11:16:02 +0200
Note that I also uncommented a section of code related to trial info. This was commented out by Jan-Mathijs in revision 3896, see https://code.google.com/p/fieldtrip/source/diff?spec=svn3896&r=3896&format=side&path=/trunk/ft_appenddata.m it is unclear why it was commented out and the comment "bugfix - #845 sampleinfo should only be initialized if required in output" does not seem to indicate that it should be commented out.