octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #8827] mapping package: new functions wrap


From: anonymous
Subject: [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi
Date: Mon, 28 Dec 2015 21:09:42 +0000
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:43.0) Gecko/20100101 Firefox/43.0

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

Hello Philip, 

thank you very much for your functions. I had a look on you code and I've
tested the performance of both algorithms. My approach seems to by faster.
I've updated my code with your template including the tests. 

In order to increase the performance of all the functions I've implemented the
algorithm in all of them instead of calling wrapTo2Pi with angle scaling on
the others. This approach works better. 

Regarding the tests, I've located an error in the 18th component in you
wrapTo180.m. ¿Could you confirm it?, it is stated that the output or
wrapTo180(180) = -180 and it should be 180. Am I correct?

Feel free to use your algorithm or mine, both work correctly. 

Regards, 

Òscar

(file #35879)
    _______________________________________________________

Additional Item Attachment:

File name: WrapTo.tar.gz                  Size:1 KB


    _______________________________________________________

Reply to this item at:

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

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]