Back to the main page.

Bug 3093 - ft_artifact_zvalue>redraw_cb looks for cfg.datafile instead of the input data

Reported 2016-03-09 23:50:00 +0100
Modified 2016-06-14 16:14:47 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P5 normal
Assigned to: Robert Oostenveld
Depends on:
See also:

Teresa Madsen - 2016-03-09 23:50:08 +0100

I am getting these error messages: Reference to non-existent field 'datafile'. Error in ft_artifact_zvalue>redraw_cb (line 906) data = ft_read_data(cfg.datafile, 'header', hdr, 'begsample', trl(trlop,1), 'endsample', trl(trlop,2), 'chanindx', sgnind, 'checkboundary', strcmp(cfg.continuous,'no')); Error in ft_artifact_zvalue (line 498) redraw_cb(h); Error in FT_preprocFear_chunks_TEM (line 401) [~, artifact.thetafreq{t}] = ft_artifact_zvalue(cfg,data); The line where the error is thrown is under "else" for "if ~isempty(" -- meaning it's trying to reload the data from the file when it can't find it in opt. Since I am including the data as an input to ft_artifact_zvalue, I'm not sure why it's not showing up in in this subfunction. Tracking it back into the main ft_artifact_zvalue function, I think this bit at line 478 could be the problem: if hasdata = {}; else = data; end Isn't that backwards? Adding a tilde to hasdata seems to fix it. Was this changed recently? I've only had this problem after downloading the latest version to troubleshoot another bug yesterday.

Robert Oostenveld - 2016-03-10 10:11:39 +0100

(In reply to Teresa Madsen from comment #0) Thanks for reporting. It has to do with related to where I made a change to deal better with cfg.inputfile input. So actually there are three options to consider, where 3 was not working correctly on all MATLAB versions: 1) cfg.dataset 2) data provided as input arg 3) data read from cfg.inputfile I agree to your proposed solution. This should fix it: 130-229-40-210> git commit ft_artifact_zvalue.m [master 609bad9] FIX - hasdata should be ~hasdata, see and thanks to Theresa 1 file changed, 1 insertion(+), 1 deletion(-) 130-229-40-210> git push upstream master X11 forwarding request failed on channel 0 You're about to push master, is that what you intended? [y|n] y Counting objects: 5, done. Delta compression using up to 4 threads. Compressing objects: 100% (3/3), done. Writing objects: 100% (3/3), 382 bytes | 0 bytes/s, done. Total 3 (delta 2), reused 0 (delta 0) To bf61f23..609bad9 master -> master

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