Back to the main page.

Bug 2966 - ft_databrowser not creating cfg.headerfile for EDF

Status CLOSED FIXED
Reported 2015-09-25 08:08:00 +0200
Modified 2016-06-14 16:14:53 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P5 normal
Assigned to: Robert Oostenveld
URL:
Tags:
Depends on:
Blocks:
See also:

Robert Oostenveld - 2015-09-25 08:08:57 +0200

Stefan wrote on http://mailman.science.ru.nl/pipermail/fieldtrip/2015-September/009649.html: I tried plotting an edf file. At line 325 the function ft_read_header seems to expect a field in the config structure that does not exist. Do you know if I am doing something wrong or if this is a bug? I am using the current field trip version downloaded from github. Thank you! Stephan cfg=[]; cfg.viewmode = 'vertical'; cfg.continuous ='yes'; cfg.datafile = ‘/data/test.edf'; ft_databrowser(cfg) Reference to non-existent field 'headerformat'. Error in ft_databrowser (line 325) hdr = ft_read_header(cfg.headerfile, 'headerformat', cfg.headerformat);


Robert Oostenveld - 2015-09-25 08:44:16 +0200

(In reply to Robert Oostenveld from comment #0) This works (and is according to documentation) cfg = [] cfg.dataset = '0601_s.edf'; ft_databrowser(cfg) This fails cfg = [] cfg.datafile = '0601_s.edf' ft_databrowser(cfg) Reference to non-existent field 'headerformat'. Error in ft_databrowser (line 325) hdr = ft_read_header(cfg.headerfile, 'headerformat', cfg.headerformat);


Robert Oostenveld - 2015-09-25 08:53:03 +0200

there is no reason it should not work. I checked and the sequence ft_databrowser->ft_checkconfig->dataset2files is correctly in place, but dataset2files was not dealing with this particular case properly. This improves it. mac011> svn commit utilities/ft_checkconfig.m Sending utilities/ft_checkconfig.m Transmitting file data . Committed revision 10690.


Robert Oostenveld - 2016-06-14 16:14:53 +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.