[CHANGESET]: Statistics function incorrectly computing median

Ben Abbott bpabbott at mac.com
Sun Mar 9 12:23:49 CDT 2008


On Mar 9, 2008, at 1:15 PM, Jaroslav Hajek wrote:

> On Sun, Mar 9, 2008 at 5:33 PM, Ben Abbott <bpabbott at mac.com> wrote:
>>
>> On Mar 9, 2008, at 10:43 AM, Jaroslav Hajek wrote:
>>
>>> Hi Ben,
>>>
>>> in case you're interested, I've created a loop-free version of
>>> __quantile__ (attached as __quantile_vec__). It appears to be  
>>> several
>>> times faster than your version. OTOH, as you've expected, the memory
>>> consumption is certainly higher.
>>
>> I just glanced over the __quantile_vec__.m and noticed that the test
>> scripts refer to "quantile_vec__"
>>
>> I tried changing all references to __quantile_vec__, but the tests
>> failed.
>>
>> Ben
>>
>
> I'm sorry, I've ignored the tests so far (the name just got changed by
> search & replace).
> Looking at them now, they're bound to fail, as the __quantile_vec__ is
> supposed to operate on columns of a matrix, and you pass a row vector
> for x everywhere.
> I'll correct the tests.

Great. When this is done, your vectorized version will be a  
replacement for quantile.m correct?

Ben



More information about the Bug-octave mailing list