[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/
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, (continued)
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, anonymous, 2015/12/22
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/22
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/24
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Andreas Weber, 2015/12/24
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Carnë Draug, 2015/12/24
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/24
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/24
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/25
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, anonymous, 2015/12/28
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/28
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi,
anonymous <=
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/29
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/29
- [Octave-patch-tracker] [patch #8827] mapping package: new functions wrapTo 360, 2Pi, 180, Pi, Philip Nienhuis, 2015/12/29