Back to the main page.
Bug 2712 - cfg.trl option doesn't seem to work in ft_databrowser
Status | RESOLVED WORKSFORME |
Reported | 2014-09-29 16:21:00 +0200 |
Modified | 2021-09-16 13:53:44 +0200 |
Product: | FieldTrip |
Component: | core |
Version: | unspecified |
Hardware: | PC |
Operating System: | Windows |
Importance: | P5 normal |
Assigned to: | Roemer van der Meij |
URL: | |
Tags: | |
Depends on: | |
Blocks: | 2314 |
See also: |
Johanna - 2014-09-29 16:21:24 +0200
The documentation of ft_databrowser mentions use of cfg.trl, only for use with trial-based data. I'm entering trial-based timelock data (output of ft_timelockanalysis with keeptrials='yes') and yet it seems to ignore my use of cfg.trl but instead reverts to timelock.sampleinfo. I looked in the code of ft_databrowser and can't find any place where cfg.trl is used. Nor is there a default set for it in the initial lines of 'set the defaults' (lines 156-182). Is this cfg option obsolete (in which case, should be removed from documentation) or should it work? Am I calling it wrongly? cfg=[]; cfg.trl=timelock.sampleinfo(2:end,:); ft_databrowser(cfg,timelock) Thank you, Johanna
Jan-Mathijs Schoffelen - 2016-07-04 17:05:39 +0200
I can confirm this. @Roemer: has this worked at some point? @Roemer: should it still work? If not, I suggest to remove it from the documentation altogether, because indeed cfg.trl does not seem to be interrogated (anymore) in the function
Roemer van der Meij - 2016-07-04 17:29:22 +0200
I never used/worked on cfg.trl, so no idea. My guess would have been that it would be used if you only give the browser a cfg (since it's cfg.trl not cfg.trials), but a quick check told me it is ignored in that case as well. No idea why. Removal seems solid to me.