Back to the main page.
Bug 2051 - ft_math does not work on secondary fields
| Status | CLOSED FIXED |
| Reported | 2013-03-13 14:35:00 +0100 |
| Modified | 2014-07-03 17:24:12 +0200 |
| Product: | FieldTrip |
| Component: | core |
| Version: | unspecified |
| Hardware: | PC |
| Operating System: | Windows |
| Importance: | P3 normal |
| Assigned to: | Robert Oostenveld |
| URL: | |
| Tags: | |
| Depends on: | |
| Blocks: | |
| See also: | http://bugzilla.fcdonders.nl/show_bug.cgi?id=2054http://bugzilla.fcdonders.nl/show_bug.cgi?id=2625 |
Jörn M. Horschig - 2013-03-13 14:35:41 +0100
e.g. does not work on source.avg.pow. As test data, you can use test_beamforming_extended (which is fixed again) and use source_exp and source_bsl to create source_diff
Robert Oostenveld - 2013-03-13 14:59:49 +0100
I can confirm the problem: Error using ft_math (line 67) the requested parameter is not present in the data Error in test_bug2051 (line 11) powlog = ft_math(cfg, source_coh_lft);
Robert Oostenveld - 2013-03-13 17:35:31 +0100
r7661 | roboos | 2013-03-13 17:33:48 +0100 (Wed, 13 Mar 2013) | 2 lines enhancement - added full provenance to ft_selectdata, implemented support for (new-style) source data in ft_selectdata_new, use rollback_provenance to keep provenance and cfg intact when doing an excursion from ft_math to ft_selectdata, added a 2013x version in ft_datatype_source (to reflect the new-style, still to be discussed with jansch), added a test script for http://bugzilla.fcdonders.nl/show_bug.cgi?id=2053
Robert Oostenveld - 2013-03-13 17:38:31 +0100
I though it would be an easy one, but it took me 2.5 hours :-( it turned out that it was not a simple case of using getsubfield, but ft_selectdata had to be improved as well. @Jorn, can you check out the test script and try it on your own data? I suspect that you'll be a bit surprised about it also converting the source representation. Let's discuss this in the next FT meeting (in 2 weeks for me).
Robert Oostenveld - 2013-05-17 17:32:23 +0200
roboos@mentat001> svn commit -m "bugfix - support source structures in ft_math by converting them to new style, see http://bugzilla.fcdonders.nl/show_bug.cgi?id=2051" Sending ft_math.m Sending utilities/ft_selectdata_new.m Transmitting file data .. Committed revision 8133.