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

[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: Tue, 14 Jul 2020 17:12:19 -0400 (EDT)
User-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.121 Safari/537.36

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

(mapping pkg maintainer here)
Thanks for your contribution.

Indeed I recently added the angl2str and str2angle functions, primarily as
some prerequisite for projection functions.
AFAICS they're Matlab-compatible, except for a few rarely used packed formats
that were hard to parse using regexps. If you have a good solution for those
I'm definitely interested.

I'm a bit puzzled by your remarks about the round() function. If you think
that needs amendments you should open a separate bug report for that, but as
round() is a core function you'd better have convincing argumentation.
Adding another round.m function to the mapping package meant to shadow the
core Octave one to support just one or two individual functions might going to
be a hard sell as well. There are probably several other functions in the
mapping package or in its dependencies or dependers that rely on the current
round() behavior; let alone core Octave functions.
There are workarounds though. More on that later on:

BTW I have very little time for Octave until the end of July so no hurry as
far as I'm concerned. I'll probably look into this early August.

    _______________________________________________________

Reply to this item at:

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

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




reply via email to

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