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

[Philip Nienhuis]

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
with.
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:

  <https://savannah.gnu.org/patch/?9953>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/