[Octave-bug-tracker] [bug #54567] median(uint8) gives bad results

[Rik]

Follow-up Comment #8, bug #54567 (project octave):

Even though it makes for clear code to call mean(), we should not do it.  This
is the mean of only two numbers and the overhead involved in calling an Octave
function is significant.  For core Octave we do prioritize speed and
efficiency to some degree over readability of code.  So the calculation should
be be done in median.m, and probably needs an exception case for (u)int64
values.  I can review any amended changeset.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?54567>

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