Back to the main page.

Bug 1110 - ft_postamble attempts to address 'cfg' in fieldtrip2spss while this variable is never put in or created

Status CLOSED FIXED
Reported 2011-11-03 14:26:00 +0100
Modified 2019-08-10 12:41:43 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Linux
Importance: P1 normal
Assigned to: Robert Oostenveld
URL:
Tags:
Depends on:
Blocks:
See also:

Arjen Stolk - 2011-11-03 14:26:21 +0100

??? Undefined function or variable 'cfg'. Error in ==> ft_postamble_trackconfig at 12 if isfield(cfg, 'outputfile') Error in ==> ft_postamble at 27 evalin('caller', ['ft_postamble_' cmd]); Error in ==> fieldtrip2spss at 55 ft_postamble trackconfig Instead of the trackconfig option, what option should be here?


Robert Oostenveld - 2011-11-03 14:37:42 +0100

it should have been the callinfo postamble I fixed it


Robert Oostenveld - 2011-11-11 11:19:48 +0100

I closed all bugs that were recently resolved.


Arjen Stolk - 2017-11-14 18:02:39 +0100

The same issue occurs with callinfo (ft_preamble/ft_postamble callinfo). Tried changing it to 'debug' as callinfo seems no longer supported, but that didn't help much. Sorry, never really understood the intrinsics of preamble and postamble: Undefined function or variable 'cfg'. Error in ft_postamble_debug (line 15) if isfield(cfg, 'verbose') && ischar(cfg.verbose) Error in ft_postamble (line 86) evalin('caller', full_cmd); Error in fieldtrip2spss (line 54) ft_postamble debug test script: labels = {'ones', 'twos', 'threes'} data = [1 2 3; 1 2 3; 1 2 3] filename = 'tessie.txt' fieldtrip2spss(filename, labels, data)


Robert Oostenveld - 2017-11-14 18:19:13 +0100

I don't think that this ft_defaults ft_preamble callinfo and ft_postamble callinfo belong in this function. Can you remove them and send a PR?


Arjen Stolk - 2017-11-14 18:24:46 +0100

(In reply to Robert Oostenveld from comment #4) Ok, see https://github.com/fieldtrip/fieldtrip/pull/580


Arjen Stolk - 2017-11-14 18:25:11 +0100

fixed


Robert Oostenveld - 2019-08-10 12:35:30 +0200

This closes a whole series of bugs that have been resolved (either FIXED/WONTFIX/INVALID) for quite some time. If you disagree, please file a new issue on https://github.com/fieldtrip/fieldtrip/issues.


Robert Oostenveld - 2019-08-10 12:41:43 +0200

This closes a whole series of bugs that have been resolved (either FIXED/WONTFIX/INVALID) for quite some time. If you disagree, please file a new issue on https://github.com/fieldtrip/fieldtrip/issues.