Back to the main page.

Bug 2099 - ft_preproc_padding fails for padtype mirror

Status CLOSED FIXED
Reported 2013-04-10 16:51:00 +0200
Modified 2014-03-12 12:21:44 +0100
Product: FieldTrip
Component: preproc
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P3 normal
Assigned to: Jörn M. Horschig
URL:
Tags:
Depends on:
Blocks:
See also:

Arjen Stolk - 2013-04-10 16:51:59 +0200

In line 79 it states the variable 'padlength'. This variable is never specified. Likely, 'postpadlength' was intended here?


Jörn M. Horschig - 2013-04-10 17:12:39 +0200

yep exactly... wondering how that could have happened... ;) gonna change it in a bit


Jörn M. Horschig - 2013-04-10 17:14:23 +0200

665 $ svn ci ft_preproc_padding.m -m "bugfix #2099 - variable name corrected" Sending ft_preproc_padding.m Transmitting file data . Committed revision 7774.


Arjen Stolk - 2013-04-10 18:16:58 +0200

svn :o :D


Arjen Stolk - 2013-04-10 21:01:26 +0200

http://code.google.com/p/fieldtrip/source/diff?spec=svn7774&r=7774&format=side&path=/trunk/preproc/ft_preproc_padding.m I think the lines 44-46 are not ncessary. In fact they cause another error ( http://code.google.com/p/fieldtrip/source/detail?r=7774 ). Namely, the variable padlength is never specified, also not in the function call: function [dat] = ft_preproc_padding(dat, padtype, prepadlength, postpadlength) :)


Arjen Stolk - 2013-04-10 21:04:49 +0200

Just for clarity; deleting those lines 44-46 should solve the 'new' problem. The 'old' problem you have solved already.


Robert Oostenveld - 2013-04-10 21:22:02 +0200

I noticed a large number of test scripts failing on the dashboard, which turned out to be due to an unknown variable padlength in ft_preproc_padding. Apparently something went wrong here. I fixed the previous fix. roboos@mentat001> svn commit Sending preproc/ft_preproc_padding.m Transmitting file data . Committed revision 7777.


Arjen Stolk - 2013-04-10 21:24:35 +0200

Hehe, you just beat me to it. bash-4.1$ svn commit ft_preproc_padding.m Sending ft_preproc_padding.m Transmitting file data .svn: Commit failed (details follow): svn: File '/trunk/preproc/ft_preproc_padding.m' is out of date svn: Your commit message was left in a temporary file: svn: '/home/action/arjsto/fieldtrip-svn/preproc/svn-commit.tmp'


Arjen Stolk - 2013-04-10 21:27:29 +0200

Just tested the new ft_preproc_padding and it works perfectly fine now.


Arjen Stolk - 2013-04-10 21:27:51 +0200

see above