Mercurial > octave
diff scripts/statistics/base/kurtosis.m @ 21309:7fbecef105ca
Allow statistics functions to work over non-existent dimension (bug #33523).
* var.m, center.m, kurtosis.m, mean.m, meansq.m, median.m, moment.m,
skewness.m, std.m: Remove check for input dimension being within the range
of ndims (x). Add BIST tests for new behavior and update input validation
tests.
| author | Rik <rik@octave.org> |
|---|---|
| date | Fri, 19 Feb 2016 21:27:03 -0800 |
| parents | c53bfd6d8e08 |
| children | bac0d6f07a3e |
line wrap: on
line diff
--- a/scripts/statistics/base/kurtosis.m Fri Feb 19 09:40:59 2016 -0800 +++ b/scripts/statistics/base/kurtosis.m Fri Feb 19 21:27:03 2016 -0800 @@ -105,12 +105,12 @@ ## Find the first non-singleton dimension. (dim = find (sz > 1, 1)) || (dim = 1); else - if (! (isscalar (dim) && dim == fix (dim)) || ! (1 <= dim && dim <= nd)) + if (! (isscalar (dim) && dim == fix (dim) && dim > 0)) error ("kurtosis: DIM must be an integer and a valid dimension"); endif endif - n = sz(dim); + n = size (x, dim); sz(dim) = 1; x = center (x, dim); # center also promotes integer, logical to double @@ -140,6 +140,7 @@ %!assert (kurtosis ([-3, 0, 1]) == kurtosis ([-1, 0, 3])) %!assert (kurtosis (ones (3, 5)), NaN (1, 5)) +%!assert (kurtosis (1, [], 3), NaN) %!assert (kurtosis ([1:5 10; 1:5 10], 0, 2), 5.4377317925288901 * [1; 1], 8 * eps) %!assert (kurtosis ([1:5 10; 1:5 10], 1, 2), 2.9786509002956195 * [1; 1], 8 * eps) @@ -165,5 +166,4 @@ %!error <DIM must be an integer> kurtosis (1, [], ones (2,2)) %!error <DIM must be an integer> kurtosis (1, [], 1.5) %!error <DIM must be .* a valid dimension> kurtosis (1, [], 0) -%!error <DIM must be .* a valid dimension> kurtosis (1, [], 3)