[Top][All Lists]

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

[Octave-patch-tracker] [patch #9953] [octave forge] (mapping) angl2str

From: Philip Nienhuis
Subject: [Octave-patch-tracker] [patch #9953] [octave forge] (mapping) angl2str
Date: Wed, 2 Sep 2020 14:16:16 -0400 (EDT)
User-agent: Mozilla/5.0 (Windows NT 6.1; rv:52.0) Gecko/20100101 Firefox/52.0

Follow-up Comment #13, patch #9953 (project octave):

I've brushed up my angl2str.m version, renamed it to angltostr.m and added a
provisional angl2str.m wrapper that for the time being calls angltostr.m
behind the scenes.
Hopefully we can soon drop the wrapper and get your version in place then.

My motives are that IMO Matlab's angl2str is actually little more than a
kludge. It doesn't fit my needs and needs significant pre- and post processing
to get acceptable (IMO) results [*]. angltostr.m does a better job, although I
already saw some issues :-)

[*] Some mentioned in comment #4: no wrapping into [-180, 180] or maybe [0,
360], or [-90 90] for latitudes. I've never seen use cases for value outside
those ranges.
In addition Matlab's help explicitly mentions input in degrees only so I'd
expect radians output to be converted from degrees, but actually there's no
conversion at all, angl2str actually only does formatting.
char arrays are also a pain often, cellstr arrays are much easier to work
I'll probably file a bug report at TMW, but in general the outcome of that is
rarely satisfying IMO.


Reply to this item at:


  Message sent via Savannah

reply via email to

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