Back to the main page.

Bug 2097 - rollback_provenance transforms strings to cells? (ft_math dimord checking faulty)

Status CLOSED FIXED
Reported 2013-04-09 10:53:00 +0200
Modified 2019-08-10 12:33:09 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P3 normal
Assigned to:
URL:
Tags:
Depends on:
Blocks:
See also:

Jörn M. Horschig - 2013-04-09 10:53:35 +0200

line 77: if isfield(varargin{1}, [cfg.parameter 'dimord']) dimord = varargin{1}.([cfg.parameter 'dimord']); (...) While this should work in theory, the problem is that a line before rollback_provenance is called, which transforms cfg.parameter to a cell-array. Subsequently, the isfield command checks for the presence of cfg.parameter *and* 'dimord', thus returns a vector with two elements and the varargin{1}.([cfg.parameter 'dimord']) is not allowed syntax (??? Argument to dynamic structure reference must evaluate to a valid field name.) Steps to do: - Check why rollback provenance is doing this


Jörn M. Horschig - 2013-04-09 10:56:28 +0200

For now, I implemented a dirty workaround: 785 $ svn ci ft_math.m -m "workaround #2097 - undoing cell conversion for cfg.parameter" Sending ft_math.m Transmitting file data . Committed revision 7772.


Jan-Mathijs Schoffelen - 2016-06-07 13:15:41 +0200

This seems to have been addressed.


Robert Oostenveld - 2019-08-10 12:33:09 +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.