Back to the main page.
Bug 1233 - typo in read_eeglabdata?
| Status | CLOSED FIXED |
| Reported | 2011-12-14 10:38:00 +0100 |
| Modified | 2012-08-23 14:02:06 +0200 |
| Product: | FieldTrip |
| Component: | fileio |
| Version: | unspecified |
| Hardware: | PC |
| Operating System: | Mac OS |
| Importance: | P3 normal |
| Assigned to: | Jörn M. Horschig |
| URL: | |
| Tags: | |
| Depends on: | |
| Blocks: | |
| See also: |
Jan-Mathijs Schoffelen - 2011-12-14 10:38:53 +0100
Dear Fieldtrip developers, I encountered an error warning when reading in an eeglab *.set - file for preprocessing (version fieldtrip-20111206), where the call to ft_preprocessing says ??? Undefined function or variable "hdr". Error in ==> read_eeglabdata at 52 if isempty(hdr) Shouldn't code lines 52 to 54 read if isempty(header) header = read_eeglabheader(filename); end instead of if isempty(hdr) hdr = read_eeglabheader(filename); end Not really a problem, just to bring it to your attention... Best regards, Gregor
Jörn M. Horschig - 2011-12-15 13:36:05 +0100
506 $ svn ci read_eeglabdata.m -m "bugfix - changed hdr to header - thx Gregor" Sending read_eeglabdata.m Transmitting file data . Committed revision 5049.
Boris Reuderink - 2012-01-03 15:26:11 +0100
Thanks for fixing this Jörn. Could you put a short and simple test script in the repository, so that we can automatically test that this bug does not happen again? Boris
Jörn M. Horschig - 2012-01-06 15:18:14 +0100
Hey Boris, I don't have any eeglabdata, sorry... Maybe someone else is using eeglab?
Boris Reuderink - 2012-01-10 12:56:31 +0100
Gregor offered to provide a EEGlab file.