Re: octave findpeaks

[Juan Pablo Carbajal]

On Thu, Dec 5, 2013 at 11:00 AM, JuanPi <address@hidden> wrote:
> On Wed, Dec 4, 2013 at 7:01 PM, Ricardo Santiago Mozos
> <address@hidden> wrote:
>> Dear Juan Pablo,
>>
>> I found a bug in findpeaks. I attach the data.
>>
>> load data.mat
>> figure,plot(x),[a,b]=findpeaks(x,"DoubleSided");hold on, plot(b,a,'x'),hold
>> off
>>
>> error: findpeaks: subscript indices must be either positive integers or
>> logicals
>> error: called from:
>> error:   /usr/share/octave/packages/signal-1.2.2/findpeaks.m at line 126,
>> column 5
>>
>> Best,
>> Ricardo.
>
> Thanks for the report, I will work on it.
> In the future please use the savannah bug tracker
> https://savannah.gnu.org/bugs/?group=octave
> so the whole community is updated.
>
>
> --
> JuanPi Carbajal
> -----
> "Ave Imperator, moritvri te salvtant"
> Naumachiarii, AD 52

The bug is not present in the current develop version of the function.
Get it from here
https://sourceforge.net/p/octave/signal/ci/default/tree/inst/findpeaks.m

and replace the function in your package "inst" directory, or wait
until signal 1.3.0 is released