Back to the main page.

Bug 2233 - ft_rejectartifact: fails to execute artifact_xxx since the compat directory has been reorganized

Status CLOSED FIXED
Reported 2013-08-06 13:08:00 +0200
Modified 2013-10-26 18:03:28 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P3 normal
Assigned to: Jan-Mathijs Schoffelen
URL:
Tags:
Depends on:
Blocks:
See also:

Jan-Mathijs Schoffelen - 2013-08-06 13:08:51 +0200

Backward compatibility for non-prefixed ft-functions has been removed recently, causing ft_rejectartifact to crash in line 257 reported by Izabela. solution: add ft_ to the eval statement.


Jan-Mathijs Schoffelen - 2013-08-06 13:10:00 +0200

fixed upon revision 8380. Note: are there any other places in FT that try to feval a non-prefixed function?


Robert Oostenveld - 2013-08-06 13:50:22 +0200

(In reply to comment #1) There are trialfuns, browsefuns and low-level freq stuff. I think that those have all been dealt with. Looking at ft_definetrial, I see at line 140 cfg.trialfun = ft_getuserfun(cfg.trialfun, 'trialfun'); See private/ft_getuserfun.m I think the desired implementation is to use ft_getuserfun also for the feval in ft_rejectartifact.


Jan-Mathijs Schoffelen - 2013-09-24 16:10:51 +0200

This seems to have been implemented. Setting this to fixed.