[Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi

[Carnë Draug]

Follow-up Comment #12, patch #8827 (project octave):

This has been mentioned before on comment #2.

Can you please add some tests for this functions? The multiple cases by Philip
Nienhuis would work nicely.  For example, at the end of wrapToPi, you can add
the tests:


%!assert (wrapToPi (pi * [-3:.5:3]),  pi * [-1 -.5 0 -.5 -1 -.5  0 .5 1 .5 0
.5 1], eps)
%!assert (wrapToPi ([Inf NaN pi]),  [XX XX pi], eps)
## Test more than one dimensional vector
%!assert (wrapToPi (pi .* [-2 -1/2 0; 2 1/3 5/3]),  pi* [0 -1/2 0; 1 1/3 2/3],
eps)


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8827>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/