Back to the main page.

Bug 2538 - update all calls to ft_selectdata_old to use the common (newer) implementation

Reported 2014-04-15 19:06:00 +0200
Modified 2016-01-15 09:37:18 +0100
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P3 normal
Assigned to: Jan-Mathijs Schoffelen
Depends on:
Blocks: 1021
See also:

Robert Oostenveld - 2014-04-15 19:06:57 +0200

the following functions use old syntax with multiple input arguments. ft_appendsource.m ft_freqstatistics.m ft_freqstatistics.m ft_multiplotER.m ft_multiplotER.m ft_multiplotER.m ft_singleplotER.m ft_singleplotER.m ft_sourcestatistics.m

Robert Oostenveld - 2014-04-15 19:07:41 +0200

the following functions use the old syntax for selecting trials in a single data structure. ft_channelnormalise.m ft_channelrepair.m ft_componentanalysis.m ft_connectivityanalysis.m ft_denoise_pca.m ft_denoise_pca.m ft_denoise_synthetic.m ft_freqanalysis.m ft_freqdescriptives.m ft_megplanar.m ft_megrealign.m ft_preprocessing.m ft_redefinetrial.m ft_rejectvisual.m ft_resampledata.m ft_scalpcurrentdensity.m ft_timelockanalysis.m

Robert Oostenveld - 2014-04-15 19:12:54 +0200

the following functions have another use of the old implementation (foilim, toilim, channels, jackknife?). ft_connectivityanalysis.m ft_freqanalysis.m ft_freqdescriptives.m ft_qualitycheck.m

Jan-Mathijs Schoffelen - 2014-04-15 20:33:59 +0200

Shoot. Forgot about Jackknife altogether. What do you think, is anybody using that functionality?

Robert Oostenveld - 2014-06-12 19:36:48 +0200

mac011> svn commit ft_selectdata*m ft_selectdata_newer.m Sending ft_selectdata_new.m Deleting ft_selectdata_newer.m Sending ft_selectdata_old.m Transmitting file data .. Committed revision 9620. I removed the newer implementation, it has been fused with the proper one. I also removed the end-user documentation for old and new version, instead it now states that it is deprecated and that ft_selectdata is to be used.

Jan-Mathijs Schoffelen - 2015-02-12 09:22:04 +0100

As of today, the following high-level ft-functions still use the old version: ft_appendsource ft_connectivityanalysis ft_freqdescriptives ft_multiplotER ft_qualitycheck ft_singleplotER

Jan-Mathijs Schoffelen - 2015-02-12 09:23:38 +0100

Some of the functions mentioned in the previous comment are straightforward, the 'jackknife' option in the old implementation is used in ft_connectivityanalysis and ft_freqdescriptives (for the latter it is easy to solve).

Jan-Mathijs Schoffelen - 2015-02-12 09:28:39 +0100

$ svn commit -m "enhancement - use ft_selectdata 'new'" ft_qualitycheck.m Sending ft_qualitycheck.m Transmitting file data . Committed revision 10220.

Jan-Mathijs Schoffelen - 2015-09-14 10:22:45 +0200

ft_selectdata 'new' is now also used in ft_singleplotER.m and ft_multiplotER.m The only occurrences in high-level FT-functions are now: ft_appendsource ft_freqdescriptives and ft_connectivityanalysis (for jackknifing).

Jan-Mathijs Schoffelen - 2015-09-15 11:56:18 +0200

ft_freqdescriptives does the jackknifing inline, instead of relying on ft_selectdata_old, as of revision 10670, as discussed in the FT-meeting.

Jan-Mathijs Schoffelen - 2015-09-15 16:08:02 +0200

ft_connectivityanalysis now also only uses ft_selectdata (new). (revision 10672)

Jan-Mathijs Schoffelen - 2015-09-15 16:41:53 +0200

Also removed call to ft_selectdata (old) in ft_appendsource. This for sure breaks this function, but I think that this one didn't work to begin with. I suggest to follow this up elsewhere. This bug can be closed I think.

Robert Oostenveld - 2015-09-15 16:47:47 +0200

(In reply to Jan-Mathijs Schoffelen from comment #11) whohoo!