[Octave-bug-tracker] [bug #36718] moment, skewness, and kurtosis differences with Matlab

[Julien Bect]

Follow-up Comment #13, bug #36718 (project octave):

Rik,

Please find attached three patches for skewness.m :

a) Your coding style fixes make some test fail on my computer (that's because
you replaced some explicite tolerance values by eps). I simply replace eps by
2 * eps.

b) Because of some changes you made in input argument checking, weird (non
scalar) values of flag were able to go though undetected. I fixed that in a
manner, I hope, you will find in agreement with Octave core coding
conventions.

c) Get rid of a "division by zero" warning that I hadn't noticed before.

@++
Julien


(file #29306, file #29307, file #29308)
    _______________________________________________________

Additional Item Attachment:

File name: jbPatchSkewness5a.patch        Size:1 KB
File name: jbPatchSkewness5b.patch        Size:1 KB
File name: jbPatchSkewness5c.patch        Size:0 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?36718>

_______________________________________________
  Message posté via/par Savannah
  http://savannah.gnu.org/